Get rid of most instance of 'foo != NULL'

core/src/fpdfapi/fpdf_render/fpdf_render.cpp

Index: core/src/fpdfapi/fpdf_render/fpdf_render.cpp
diff --git a/core/src/fpdfapi/fpdf_render/fpdf_render.cpp b/core/src/fpdfapi/fpdf_render/fpdf_render.cpp
index 66607f76de7ac582b0f2c6e5a1ded9785c36fe2b..6b7ccfd43bb4d3fe6775b4a0be68e4568f3543a2 100644
--- a/core/src/fpdfapi/fpdf_render/fpdf_render.cpp
+++ b/core/src/fpdfapi/fpdf_render/fpdf_render.cpp
@@ -543,7 +543,7 @@ FX_BOOL CPDF_RenderStatus::ProcessPath(CPDF_PathObject* pPathObj,
                              m_curBlend);
 }
 CPDF_TransferFunc* CPDF_RenderStatus::GetTransferFunc(CPDF_Object* pObj) const {
-  ASSERT(pObj != NULL);
+  ASSERT(pObj);
   CPDF_DocRenderData* pDocCache = m_pContext->m_pDocument->GetRenderData();
   if (!pDocCache) {

[Tom Sepez, 2015/12/14 19:14:08]

nit: use ? oper.

[Lei Zhang, 2015/12/15 01:38:32]

Done.

     return NULL;
@@ -1355,7 +1355,7 @@ FX_BOOL CPDF_ScaledRenderBuffer::Initialize(CPDF_RenderContext* pContext,
                                             const CPDF_PageObject* pObj,
                                             const CPDF_RenderOptions* pOptions,
                                             int max_dpi) {
-  FXSYS_assert(pRect != NULL);
+  FXSYS_assert(pRect);

[Tom Sepez, 2015/12/14 21:10:27]

meh. Only called in one place with pRect as &local, change to pass by const ref
instead?

[Lei Zhang, 2015/12/15 01:38:32]

Done.

   m_pDevice = pDevice;
   if (m_pDevice->GetDeviceCaps(FXDC_RENDER_CAPS) & FXRC_GET_BITS) {
     return TRUE;