diff options
| author | Lei Zhang <thestig@chromium.org> | 2015-09-22 18:28:39 -0700 |
|---|---|---|
| committer | Lei Zhang <thestig@chromium.org> | 2015-09-22 18:28:39 -0700 |
| commit | 0244d40daf39caf97f47de9982f4aac88c0c8644 (patch) | |
| tree | 3671be215431a2b9c2fd52da5d0fcddc8b7d648d | |
| parent | 5d0e8432c63a5e7a0b42fd4621c1c454ff49985e (diff) | |
| download | pdfium-0244d40daf39caf97f47de9982f4aac88c0c8644.tar.xz | |
Merge to XFA: Add missing 'return' keyword.
Discovered with experimental Clang plugin that flags temporary objects
that are immediately destroyed.
patch from issue 1359063003 at patchset 1 (http://crrev.com/1359063003#ps1)
A=mdempsky@chromium.org
TBR=mdempsky@chromium.org
Review URL: https://codereview.chromium.org/1359103002 .
(cherry picked from commit f316bfe37e2fc977376e18f7534aae9824da240a)
Review URL: https://codereview.chromium.org/1362863002 .
| -rw-r--r-- | core/src/fxge/ge/fx_ge_fontmap.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/core/src/fxge/ge/fx_ge_fontmap.cpp b/core/src/fxge/ge/fx_ge_fontmap.cpp index 3ef5c45d4c..63d8181656 100644 --- a/core/src/fxge/ge/fx_ge_fontmap.cpp +++ b/core/src/fxge/ge/fx_ge_fontmap.cpp @@ -551,7 +551,7 @@ CFX_ByteString _FPDF_LoadTableFromTTStreamFile(IFX_FileStream* pFile, } CFX_ByteString CFX_FontMapper::GetPSNameFromTT(void* hFont) { if (m_pFontInfo == NULL) { - CFX_ByteString(); + return CFX_ByteString(); } CFX_ByteString result; FX_DWORD size = m_pFontInfo->GetFontData(hFont, 0x6e616d65, NULL, 0); |