Cleanup: Do not check pointers before deleting them, part 2.

fpdfsdk/src/fpdfformfill.cpp

Index: fpdfsdk/src/fpdfformfill.cpp
diff --git a/fpdfsdk/src/fpdfformfill.cpp b/fpdfsdk/src/fpdfformfill.cpp
index 021a78aacea7250733bb5178aad0dbf9ebf8cb5a..fb15332eae9223cccb112ae52c9fda564d3c7002 100644
--- a/fpdfsdk/src/fpdfformfill.cpp
+++ b/fpdfsdk/src/fpdfformfill.cpp
@@ -229,22 +229,13 @@ DLLEXPORT void STDCALL FPDF_FFLDraw(FPDF_FORMHANDLE hHandle, FPDF_BITMAP bitmap,
 	}
 	pDevice->RestoreState();
 
-	if(options.m_pOCContext)
-	{
-		delete options.m_pOCContext;
-		options.m_pOCContext = NULL;
-	}
-	if(pContext)
-	{
-		delete pContext;
-		pContext = NULL;
-	}
-	if(pDevice)
-	{
-		delete pDevice;
-		pDevice = NULL;
-	}
+    delete options.m_pOCContext;

[Tom Sepez, 2015/07/16 17:22:44]

nit: funny indent.

[Lei Zhang, 2015/07/16 21:20:48]

merge resolved this.

+    options.m_pOCContext = NULL;
 
+    delete pContext;
+    pContext = NULL;

[Tom Sepez, 2015/07/16 17:22:44]

nit: pointless assignments?

+    delete pDevice;
+    pDevice = NULL;
 }
 
 DLLEXPORT void STDCALL FPDF_SetFormFieldHighlightColor(FPDF_FORMHANDLE hHandle, int fieldType, unsigned long color)