diff options
| author | Lei Zhang <thestig@chromium.org> | 2015-07-16 18:28:49 -0700 |
|---|---|---|
| committer | Lei Zhang <thestig@chromium.org> | 2015-07-16 18:28:49 -0700 |
| commit | 21bf242a31eff79f754c5e9c3b41c52e33483bb0 (patch) | |
| tree | 0eff85fa75ab9091254f7560b37b95408ba8975e /core/src/fxge | |
| parent | 6915e7bd9e0892f143936697c6ba233510fb13dc (diff) | |
| download | pdfium-21bf242a31eff79f754c5e9c3b41c52e33483bb0.tar.xz | |
Cleanup: Do not check pointers before deleting them, part 2.
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1195363002 .
Diffstat (limited to 'core/src/fxge')
| -rw-r--r-- | core/src/fxge/agg/src/fx_agg_driver.cpp | 4 | ||||
| -rw-r--r-- | core/src/fxge/android/fpf_skiamodule.cpp | 6 | ||||
| -rw-r--r-- | core/src/fxge/apple/fx_mac_imp.cpp | 4 | ||||
| -rw-r--r-- | core/src/fxge/win32/fx_win32_device.cpp | 4 | ||||
| -rw-r--r-- | core/src/fxge/win32/fx_win32_dwrite.cpp | 4 |
5 files changed, 6 insertions, 16 deletions
diff --git a/core/src/fxge/agg/src/fx_agg_driver.cpp b/core/src/fxge/agg/src/fx_agg_driver.cpp index 8e7846abd4..475e5f6442 100644 --- a/core/src/fxge/agg/src/fx_agg_driver.cpp +++ b/core/src/fxge/agg/src/fx_agg_driver.cpp @@ -207,9 +207,7 @@ CFX_AggDeviceDriver::~CFX_AggDeviceDriver() { delete m_pClipRgn; for (int i = 0; i < m_StateStack.GetSize(); i ++) - if (m_StateStack[i]) { - delete (CFX_ClipRgn*)m_StateStack[i]; - } + delete (CFX_ClipRgn*)m_StateStack[i]; DestroyPlatform(); } #if _FXM_PLATFORM_ != _FXM_PLATFORM_APPLE_ diff --git a/core/src/fxge/android/fpf_skiamodule.cpp b/core/src/fxge/android/fpf_skiamodule.cpp index 37bfd9cabb..296073c4e2 100644 --- a/core/src/fxge/android/fpf_skiamodule.cpp +++ b/core/src/fxge/android/fpf_skiamodule.cpp @@ -22,10 +22,8 @@ CFPF_SkiaDeviceModule::~CFPF_SkiaDeviceModule() } void CFPF_SkiaDeviceModule::Destroy() { - if (gs_pPFModule) { - delete (CFPF_SkiaDeviceModule*)gs_pPFModule; - gs_pPFModule = NULL; - } + delete (CFPF_SkiaDeviceModule*)gs_pPFModule; + gs_pPFModule = NULL; } IFPF_FontMgr* CFPF_SkiaDeviceModule::GetFontMgr() { diff --git a/core/src/fxge/apple/fx_mac_imp.cpp b/core/src/fxge/apple/fx_mac_imp.cpp index 3a9b9f9ec3..04b04b8ba8 100644 --- a/core/src/fxge/apple/fx_mac_imp.cpp +++ b/core/src/fxge/apple/fx_mac_imp.cpp @@ -100,9 +100,7 @@ void CFX_GEModule::InitPlatform() } void CFX_GEModule::DestroyPlatform() { - if (m_pPlatformData) { - delete (CApplePlatform *) m_pPlatformData; - } + delete (CApplePlatform *)m_pPlatformData; m_pPlatformData = NULL; } #endif diff --git a/core/src/fxge/win32/fx_win32_device.cpp b/core/src/fxge/win32/fx_win32_device.cpp index 899adf5b7c..b0dbc14ebc 100644 --- a/core/src/fxge/win32/fx_win32_device.cpp +++ b/core/src/fxge/win32/fx_win32_device.cpp @@ -410,9 +410,7 @@ void CFX_GEModule::InitPlatform() } void CFX_GEModule::DestroyPlatform() { - if (m_pPlatformData) { - delete (CWin32Platform*)m_pPlatformData; - } + delete (CWin32Platform*)m_pPlatformData; m_pPlatformData = NULL; } CGdiDeviceDriver::CGdiDeviceDriver(HDC hDC, int device_class) diff --git a/core/src/fxge/win32/fx_win32_dwrite.cpp b/core/src/fxge/win32/fx_win32_dwrite.cpp index 3cde3bfd8b..889a5684b7 100644 --- a/core/src/fxge/win32/fx_win32_dwrite.cpp +++ b/core/src/fxge/win32/fx_win32_dwrite.cpp @@ -262,9 +262,7 @@ FX_BOOL CDWriteExt::DwRendingString(void* renderTarget, CFX_ClipRgn* pClipRgn, F } void CDWriteExt::DwDeleteRenderingTarget(void* renderTarget) { - if (renderTarget) { - delete (CDwGdiTextRenderer*)renderTarget; - } + delete (CDwGdiTextRenderer*)renderTarget; } void CDWriteExt::DwDeleteFont(void* pFont) { |