Add missing 'return' keyword.
authorLei Zhang <thestig@chromium.org>
Tue, 22 Sep 2015 20:45:00 +0000 (13:45 -0700)
committerLei Zhang <thestig@chromium.org>
Tue, 22 Sep 2015 20:45:00 +0000 (13:45 -0700)
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
R=mdempsky@chromium.org

Review URL: https://codereview.chromium.org/1359103002 .

core/src/fxge/ge/fx_ge_fontmap.cpp

index d8f2046..90b3c52 100644 (file)
@@ -537,7 +537,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);