diff options
| author | Lei Zhang <thestig@chromium.org> | 2015-08-14 22:22:13 -0700 |
|---|---|---|
| committer | Lei Zhang <thestig@chromium.org> | 2015-08-14 22:22:13 -0700 |
| commit | da180e9fdd4385df024cc18046f62ca47bc74d74 (patch) | |
| tree | 931e0e64ac2cbc82e3718e43418ee6bd676da4a5 /core/src/fxcrt/fx_basic_buffer.cpp | |
| parent | 2b1a2d528469cda4e9f3e36d3c7a649e0d476480 (diff) | |
| download | pdfium-da180e9fdd4385df024cc18046f62ca47bc74d74.tar.xz | |
Merge to XFA: Don't bother checking pointers before delete[] and FX_Free().
R=tsepez@chromium.org
Review URL: https://codereview.chromium.org/1297713003 .
(cherry picked from commit cb62e7657b3a9a04142028a4e6614029a08e894b)
Review URL: https://codereview.chromium.org/1287053005 .
Diffstat (limited to 'core/src/fxcrt/fx_basic_buffer.cpp')
| -rw-r--r-- | core/src/fxcrt/fx_basic_buffer.cpp | 14 |
1 files changed, 4 insertions, 10 deletions
diff --git a/core/src/fxcrt/fx_basic_buffer.cpp b/core/src/fxcrt/fx_basic_buffer.cpp index 40269e00c0..19ebfdff42 100644 --- a/core/src/fxcrt/fx_basic_buffer.cpp +++ b/core/src/fxcrt/fx_basic_buffer.cpp @@ -13,9 +13,7 @@ CFX_BinaryBuf::CFX_BinaryBuf(FX_STRSIZE size) m_pBuffer = FX_Alloc(uint8_t, size); } CFX_BinaryBuf::~CFX_BinaryBuf() { - if (m_pBuffer) { - FX_Free(m_pBuffer); - } + FX_Free(m_pBuffer); } void CFX_BinaryBuf::Delete(int start_index, int count) { if (!m_pBuffer || start_index < 0 || start_index + count > m_DataSize) { @@ -34,9 +32,7 @@ void CFX_BinaryBuf::DetachBuffer() { m_AllocSize = 0; } void CFX_BinaryBuf::AttachData(void* buffer, FX_STRSIZE size) { - if (m_pBuffer) { - FX_Free(m_pBuffer); - } + FX_Free(m_pBuffer); m_DataSize = size; m_pBuffer = (uint8_t*)buffer; m_AllocSize = size; @@ -381,10 +377,8 @@ IFX_BufferArchive::IFX_BufferArchive(FX_STRSIZE size) : m_BufSize(size), m_pBuffer(NULL), m_Length(0) {} void IFX_BufferArchive::Clear() { m_Length = 0; - if (m_pBuffer) { - FX_Free(m_pBuffer); - m_pBuffer = NULL; - } + FX_Free(m_pBuffer); + m_pBuffer = NULL; } FX_BOOL IFX_BufferArchive::Flush() { FX_BOOL bRet = DoWork(m_pBuffer, m_Length); |