Use correctly-typed error codes for fx_codec_jpx_unittest

core/src/fxcodec/codec/fx_codec_jpx_unittest.cpp

 // Copyright 2014 PDFium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include <stdint.h>
 
 #include <limits>
 
 #include "codec_int.h"
 #include "testing/gtest/include/gtest/gtest.h"
@@ skipping 27 matching lines @@
     memset(buffer, 0xbd, sizeof(buffer));
     EXPECT_EQ(kReadError, opj_read_from_memory(buffer, 0, &dd));
     EXPECT_EQ(0xbd, buffer[0]);
 
     // Reads of nonzero size do nothing but return an error code.
     memset(buffer, 0xbd, sizeof(buffer));
     EXPECT_EQ(kReadError, opj_read_from_memory(buffer, sizeof(buffer), &dd));
     EXPECT_EQ(0xbd, buffer[0]);
 
     // writes of size 0 do nothing but return an error code.
-    EXPECT_EQ(kReadError, opj_write_from_memory(buffer, 0, &dd));
+    EXPECT_EQ(kWriteError, opj_write_from_memory(buffer, 0, &dd));
 
     // writes of nonzero size do nothing but return an error code.
-    EXPECT_EQ(kReadError, opj_write_from_memory(buffer, sizeof(buffer), &dd));
+    EXPECT_EQ(kWriteError, opj_write_from_memory(buffer, sizeof(buffer), &dd));
 
     // Skips of size 0 always return an error code.
-    EXPECT_EQ(kReadError, opj_skip_from_memory(0, &dd));
+    EXPECT_EQ(kSkipError, opj_skip_from_memory(0, &dd));
 
     // Skips of nonzero size always return an error code.
-    EXPECT_EQ(kReadError, opj_skip_from_memory(1, &dd));
+    EXPECT_EQ(kSkipError, opj_skip_from_memory(1, &dd));
 
     // Seeks to 0 offset return in error.
     EXPECT_FALSE(opj_seek_from_memory(0, &dd));
 
     // Seeks to non-zero offsets return in error.
     EXPECT_FALSE(opj_seek_from_memory(1, &dd));
 }
 
 TEST(fxcodec, DecodeDataZeroSize) {
     DecodeData dd(stream_data, 0);
     unsigned char buffer[16];
 
     // Reads of size 0 do nothing but return an error code.
     memset(buffer, 0xbd, sizeof(buffer));
     EXPECT_EQ(kReadError, opj_read_from_memory(buffer, 0, &dd));
     EXPECT_EQ(0xbd, buffer[0]);
 
     // Reads of nonzero size do nothing but return an error code.
     memset(buffer, 0xbd, sizeof(buffer));
     EXPECT_EQ(kReadError, opj_read_from_memory(buffer, sizeof(buffer), &dd));
     EXPECT_EQ(0xbd, buffer[0]);
 
     // writes of size 0 do nothing but return an error code.
-    EXPECT_EQ(kReadError, opj_write_from_memory(buffer, 0, &dd));
+    EXPECT_EQ(kWriteError, opj_write_from_memory(buffer, 0, &dd));
 
     // writes of nonzero size do nothing but return an error code.
-    EXPECT_EQ(kReadError, opj_write_from_memory(buffer, sizeof(buffer), &dd));
+    EXPECT_EQ(kWriteError, opj_write_from_memory(buffer, sizeof(buffer), &dd));
 
     // Skips of size 0 always return an error code.
-    EXPECT_EQ(kReadError, opj_skip_from_memory(0, &dd));
+    EXPECT_EQ(kSkipError, opj_skip_from_memory(0, &dd));
 
     // Skips of nonzero size always return an error code.
-    EXPECT_EQ(kReadError, opj_skip_from_memory(1, &dd));
+    EXPECT_EQ(kSkipError, opj_skip_from_memory(1, &dd));
 
     // Seeks to 0 offset return in error.
     EXPECT_FALSE(opj_seek_from_memory(0, &dd));
 
     // Seeks to non-zero offsets return in error.
     EXPECT_FALSE(opj_seek_from_memory(1, &dd));
 }
 
 TEST(fxcodec, DecodeDataReadInBounds) {
     unsigned char buffer[16];
@@ skipping 360 matching lines @@
     EXPECT_EQ(0x84, buffer[0]);
     EXPECT_EQ(0xbd, buffer[1]);
 
     // Seeking way beyond EOS is allowed, doesn't wrap, and read fails.
     memset(buffer, 0xbd, sizeof(buffer));
     EXPECT_TRUE(opj_seek_from_memory(
         std::numeric_limits<OPJ_OFF_T>::max(), &dd));
     EXPECT_EQ(kReadError, opj_read_from_memory(buffer, 1, &dd));
     EXPECT_EQ(0xbd, buffer[0]);
 }