diff options
| author | Lei Zhang <thestig@chromium.org> | 2017-06-23 23:15:07 -0700 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-06-26 21:03:03 +0000 |
| commit | 4ac12f0e49bfd1e0456a8f2314727deec8648936 (patch) | |
| tree | 9486f1402fd4cfba52a88fd27a8a31985076dc45 /xfa/fde/css/cfde_csstextbuf.cpp | |
| parent | f76e9d33b6246358d34cb476dbf197788fe49425 (diff) | |
| download | pdfium-4ac12f0e49bfd1e0456a8f2314727deec8648936.tar.xz | |
Remove a const_cast in CFDE_CSSTextBuf.
Change-Id: If6d2cce6a4185c6726df7d6b9d44d74efe0be11d
Reviewed-on: https://pdfium-review.googlesource.com/6931
Reviewed-by: Henrique Nakashima <hnakashima@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Diffstat (limited to 'xfa/fde/css/cfde_csstextbuf.cpp')
| -rw-r--r-- | xfa/fde/css/cfde_csstextbuf.cpp | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/xfa/fde/css/cfde_csstextbuf.cpp b/xfa/fde/css/cfde_csstextbuf.cpp index c408a95b03..8ccdca9fed 100644 --- a/xfa/fde/css/cfde_csstextbuf.cpp +++ b/xfa/fde/css/cfde_csstextbuf.cpp @@ -10,6 +10,7 @@ CFDE_CSSTextBuf::CFDE_CSSTextBuf() : m_bExtBuf(false), + m_pExtBuffer(nullptr), m_pBuffer(nullptr), m_iBufLen(0), m_iDatLen(0), @@ -21,7 +22,7 @@ CFDE_CSSTextBuf::~CFDE_CSSTextBuf() { void CFDE_CSSTextBuf::AttachBuffer(const wchar_t* pBuffer, int32_t iBufLen) { Reset(); - m_pBuffer = const_cast<wchar_t*>(pBuffer); + m_pExtBuffer = pBuffer; m_iDatLen = m_iBufLen = iBufLen; m_bExtBuf = true; } @@ -36,6 +37,8 @@ bool CFDE_CSSTextBuf::EstimateSize(int32_t iAllocSize) { bool CFDE_CSSTextBuf::AppendChar(wchar_t wch) { if (m_iDatLen >= m_iBufLen && !ExpandBuf(m_iBufLen * 2)) return false; + + ASSERT(!m_bExtBuf); m_pBuffer[m_iDatLen++] = wch; return true; } @@ -49,12 +52,17 @@ void CFDE_CSSTextBuf::Reset() { } int32_t CFDE_CSSTextBuf::TrimEnd() { + ASSERT(!m_bExtBuf); while (m_iDatLen > 0 && m_pBuffer[m_iDatLen - 1] <= ' ') --m_iDatLen; AppendChar(0); return --m_iDatLen; } +const wchar_t* CFDE_CSSTextBuf::GetBuffer() const { + return m_bExtBuf ? m_pExtBuffer : m_pBuffer; +} + bool CFDE_CSSTextBuf::ExpandBuf(int32_t iDesiredSize) { if (m_bExtBuf) return false; @@ -73,6 +81,7 @@ bool CFDE_CSSTextBuf::ExpandBuf(int32_t iDesiredSize) { void CFDE_CSSTextBuf::Subtract(int32_t iStart, int32_t iLength) { ASSERT(iStart >= 0 && iLength >= 0); + ASSERT(!m_bExtBuf); iLength = pdfium::clamp(iLength, 0, m_iDatLen - iStart); memmove(m_pBuffer, m_pBuffer + iStart, iLength * sizeof(wchar_t)); |