diff options
| author | brucedawson <brucedawson@chromium.org> | 2016-05-27 16:13:13 -0700 |
|---|---|---|
| committer | Commit bot <commit-bot@chromium.org> | 2016-05-27 16:13:13 -0700 |
| commit | 2f109ab836682cb465270ed303d27955db97d98f (patch) | |
| tree | 6c2dff01fcad76ae7b1e50a2672e651cc57b29c7 /xfa/fde | |
| parent | 24508dff1636d80be01497e30fccd21533fc0bde (diff) | |
| download | pdfium-2f109ab836682cb465270ed303d27955db97d98f.tar.xz | |
Avoid two C-style FX_WCHAR* casts in pdfium
FX_WCHAR* casts were hiding a number of misuses of narrow strings in
pdfium, which /analyze identified. Now that those are fixed I did a
search for other instances of (FX_WCHAR*) to see if any other bugs
were lurking. No bugs were found but one unnecessary cast and one
that should be a const_cast were identified. This change fixes
those. No behavioral change.
Review-Url: https://codereview.chromium.org/2024463002
Diffstat (limited to 'xfa/fde')
| -rw-r--r-- | xfa/fde/css/fde_csssyntax.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/xfa/fde/css/fde_csssyntax.cpp b/xfa/fde/css/fde_csssyntax.cpp index f948b58e71..4583595686 100644 --- a/xfa/fde/css/fde_csssyntax.cpp +++ b/xfa/fde/css/fde_csssyntax.cpp @@ -426,7 +426,7 @@ void CFDE_CSSTextBuf::Reset() { FX_BOOL CFDE_CSSTextBuf::AttachBuffer(const FX_WCHAR* pBuffer, int32_t iBufLen) { Reset(); - m_pBuffer = (FX_WCHAR*)pBuffer; + m_pBuffer = const_cast<FX_WCHAR*>(pBuffer); m_iDatLen = m_iBufLen = iBufLen; return m_bExtBuf = TRUE; } |