Proper dcheck call

cpcloud · cpcloud · commit 5c9292b42fa2 · 2017-11-19T17:29:16.000-05:00
diff --git a/src/parquet/encoding-internal.h b/src/parquet/encoding-internal.h
@@ -595,15 +595,15 @@ inline int DictEncoder<DType>::Hash(const typename DType::c_type& value) const {
 template <>
 inline int DictEncoder<ByteArrayType>::Hash(const ByteArray& value) const {
   if (value.len > 0) {
-    DCHECK(nullptr != value.ptr) << "Value ptr cannot be NULL";
+    DCHECK_NE(nullptr, value.ptr) << "Value ptr cannot be NULL";
   }
   return HashUtil::Hash(value.ptr, value.len, 0);
 }
 
 template <>
 inline int DictEncoder<FLBAType>::Hash(const FixedLenByteArray& value) const {
   if (type_length_ > 0) {
-    DCHECK(nullptr != value.ptr) << "Value ptr cannot be NULL";
+    DCHECK_NE(nullptr, value.ptr) << "Value ptr cannot be NULL";
   }
   return HashUtil::Hash(value.ptr, type_length_, 0);
 }

Original file line number	Diff line number	Diff line change
`@@ -595,15 +595,15 @@ inline int DictEncoder<DType>::Hash(const typename DType::c_type& value) const {`
`595`	`595`	`template <>`
`596`	`596`	`inline int DictEncoder<ByteArrayType>::Hash(const ByteArray& value) const {`
`597`	`597`	`if (value.len > 0) {`
`598`		`- DCHECK(nullptr != value.ptr) << "Value ptr cannot be NULL";`
	`598`	`+ DCHECK_NE(nullptr, value.ptr) << "Value ptr cannot be NULL";`
`599`	`599`	`}`
`600`	`600`	`return HashUtil::Hash(value.ptr, value.len, 0);`
`601`	`601`	`}`
`602`	`602`
`603`	`603`	`template <>`
`604`	`604`	`inline int DictEncoder<FLBAType>::Hash(const FixedLenByteArray& value) const {`
`605`	`605`	`if (type_length_ > 0) {`
`606`		`- DCHECK(nullptr != value.ptr) << "Value ptr cannot be NULL";`
	`606`	`+ DCHECK_NE(nullptr, value.ptr) << "Value ptr cannot be NULL";`
`607`	`607`	`}`
`608`	`608`	`return HashUtil::Hash(value.ptr, type_length_, 0);`
`609`	`609`	`}`