summaryrefslogtreecommitdiff
path: root/xfa/fde
diff options
context:
space:
mode:
authorbrucedawson <brucedawson@chromium.org>2016-05-27 16:13:13 -0700
committerCommit bot <commit-bot@chromium.org>2016-05-27 16:13:13 -0700
commit2f109ab836682cb465270ed303d27955db97d98f (patch)
tree6c2dff01fcad76ae7b1e50a2672e651cc57b29c7 /xfa/fde
parent24508dff1636d80be01497e30fccd21533fc0bde (diff)
downloadpdfium-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.cpp2
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;
}