Get rid of NULLs in core/

core/fxcodec/codec/fx_codec_jpx_unittest.cpp

Index: core/fxcodec/codec/fx_codec_jpx_unittest.cpp
diff --git a/core/fxcodec/codec/fx_codec_jpx_unittest.cpp b/core/fxcodec/codec/fx_codec_jpx_unittest.cpp
index 13b5425927dfe0d63a7a51bda3a105d59ef2b4a2..820a618f5b83d7b5bfdca0a0cf1869f5e8051a3d 100644
--- a/core/fxcodec/codec/fx_codec_jpx_unittest.cpp
+++ b/core/fxcodec/codec/fx_codec_jpx_unittest.cpp
@@ -23,7 +23,7 @@ TEST(fxcodec, DecodeDataNullDecodeData) {
   unsigned char buffer[16];
   DecodeData* ptr = nullptr;
 
-  // Error codes, not segvs, should callers pass us a NULL pointer.
+  // Error codes, not segvs, should callers pass us a nullptr pointer.

[Tom Sepez, 2016/06/02 20:09:49]

I think NULL is better in comments as it expresses the generic concept of
NULLness rather than nullptr pointer, which is redundant.

[Lei Zhang, 2016/06/07 07:33:23]

Done.

   EXPECT_EQ(kReadError, opj_read_from_memory(buffer, sizeof(buffer), ptr));
   EXPECT_EQ(kWriteError, opj_write_from_memory(buffer, sizeof(buffer), ptr));
   EXPECT_EQ(kSkipError, opj_skip_from_memory(1, ptr));