Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(449)

Side by Side Diff: fpdfsdk/src/fsdk_rendercontext.cpp

Issue 1192743004: Cleanup: Do not check pointers before deleting them. (Closed) Base URL: https://pdfium.googlesource.com/pdfium@master
Patch Set: rebase Created 5 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « fpdfsdk/src/fsdk_mgr.cpp ('k') | fpdfsdk/src/fxedit/fxet_edit.cpp » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2014 PDFium Authors. All rights reserved. 1 // Copyright 2014 PDFium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com 5 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com
6 6
7 #include "../include/fsdk_rendercontext.h" 7 #include "../include/fsdk_rendercontext.h"
8 8
9 void CRenderContext::Clear() 9 void CRenderContext::Clear()
10 { 10 {
11 m_pDevice = NULL; 11 m_pDevice = NULL;
12 m_pContext = NULL; 12 m_pContext = NULL;
13 m_pRenderer = NULL; 13 m_pRenderer = NULL;
14 m_pAnnots = NULL; 14 m_pAnnots = NULL;
15 m_pOptions = NULL; 15 m_pOptions = NULL;
16 #ifdef _WIN32_WCE 16 #ifdef _WIN32_WCE
17 m_pBitmap = NULL; 17 m_pBitmap = NULL;
18 m_hBitmap = NULL; 18 m_hBitmap = NULL;
19 #endif 19 #endif
20 } 20 }
21 21
22 CRenderContext::~CRenderContext() 22 CRenderContext::~CRenderContext()
23 { 23 {
24 » if (m_pRenderer) delete m_pRenderer; 24 delete m_pRenderer;
25 » if (m_pContext) delete m_pContext; 25 delete m_pContext;
26 » if (m_pDevice) delete m_pDevice; 26 delete m_pDevice;
27 » if (m_pAnnots) delete m_pAnnots; 27 delete m_pAnnots;
28 » if (m_pOptions->m_pOCContext) delete m_pOptions->m_pOCContext; 28 delete m_pOptions->m_pOCContext;
29 » if (m_pOptions) delete m_pOptions; 29 delete m_pOptions;
30 #ifdef _WIN32_WCE 30 #ifdef _WIN32_WCE
31 » if (m_pBitmap) delete m_pBitmap; 31 delete m_pBitmap;
32 » if (m_hBitmap) DeleteObject(m_hBitmap); 32 if (m_hBitmap) DeleteObject(m_hBitmap);
33 #endif 33 #endif
34 } 34 }
35 35
36 IFSDK_PAUSE_Adapter::IFSDK_PAUSE_Adapter(IFSDK_PAUSE* IPause ) 36 IFSDK_PAUSE_Adapter::IFSDK_PAUSE_Adapter(IFSDK_PAUSE* IPause )
37 { 37 {
38 m_IPause = IPause; 38 m_IPause = IPause;
39 } 39 }
40 40
41 FX_BOOL IFSDK_PAUSE_Adapter::NeedToPauseNow() 41 FX_BOOL IFSDK_PAUSE_Adapter::NeedToPauseNow()
42 { 42 {
43 if (m_IPause->NeedToPauseNow) 43 if (m_IPause->NeedToPauseNow)
44 { 44 {
45 return m_IPause->NeedToPauseNow(m_IPause); 45 return m_IPause->NeedToPauseNow(m_IPause);
46 }else{ 46 }else{
47 return FALSE; 47 return FALSE;
48 } 48 }
49 } 49 }
OLDNEW
« no previous file with comments | « fpdfsdk/src/fsdk_mgr.cpp ('k') | fpdfsdk/src/fxedit/fxet_edit.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698