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/fgas/font | |
| 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/fgas/font')
| -rw-r--r-- | xfa/fgas/font/fgas_stdfontmgr.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/xfa/fgas/font/fgas_stdfontmgr.cpp b/xfa/fgas/font/fgas_stdfontmgr.cpp index 3506aa2df9..2f2f9a3ecb 100644 --- a/xfa/fgas/font/fgas_stdfontmgr.cpp +++ b/xfa/fgas/font/fgas_stdfontmgr.cpp @@ -445,7 +445,7 @@ static void FX_EnumGdiFonts(CFX_FontDescriptors& fonts, FXSYS_memset(&lfFind, 0, sizeof(lfFind)); lfFind.lfCharSet = DEFAULT_CHARSET; if (pwsFaceName) { - FXSYS_wcsncpy((FX_WCHAR*)lfFind.lfFaceName, pwsFaceName, 31); + FXSYS_wcsncpy(lfFind.lfFaceName, pwsFaceName, 31); lfFind.lfFaceName[31] = 0; } EnumFontFamiliesExW(hDC, (LPLOGFONTW)&lfFind, |