Merge to XFA: Remove unused error variable from CFX_Font::AdjustMMParams
authorOliver Chang <ochang@chromium.org>
Thu, 22 Oct 2015 18:20:22 +0000 (11:20 -0700)
committerOliver Chang <ochang@chromium.org>
Thu, 22 Oct 2015 18:20:22 +0000 (11:20 -0700)
Also fixes a potential memory leak.

R=tsepez@chromium.org, thestig@chromium.org
BUG=pdfium:156

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

(cherry picked from commit de49cf173d12fe7a462168e2bf6c44971d7d28f8)

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

core/src/fxge/ge/fx_ge_text.cpp

index f2ac712..2f7641a 100644 (file)
@@ -1373,20 +1373,21 @@ void CFX_Font::AdjustMMParams(int glyph_index, int dest_width, int weight) {
     int min_param = FXFT_Get_MM_Axis_Min(FXFT_Get_MM_Axis(pMasters, 1)) / 65536;
     int max_param = FXFT_Get_MM_Axis_Max(FXFT_Get_MM_Axis(pMasters, 1)) / 65536;
     coords[1] = min_param;
-    int error = FXFT_Set_MM_Design_Coordinates(m_Face, 2, coords);
-    error = FXFT_Load_Glyph(
+    (void)FXFT_Set_MM_Design_Coordinates(m_Face, 2, coords);
+    (void)FXFT_Load_Glyph(
         m_Face, glyph_index,
         FXFT_LOAD_NO_SCALE | FXFT_LOAD_IGNORE_GLOBAL_ADVANCE_WIDTH);
     int min_width = FXFT_Get_Glyph_HoriAdvance(m_Face) * 1000 /
                     FXFT_Get_Face_UnitsPerEM(m_Face);
     coords[1] = max_param;
-    error = FXFT_Set_MM_Design_Coordinates(m_Face, 2, coords);
-    error = FXFT_Load_Glyph(
+    (void)FXFT_Set_MM_Design_Coordinates(m_Face, 2, coords);
+    (void)FXFT_Load_Glyph(
         m_Face, glyph_index,
         FXFT_LOAD_NO_SCALE | FXFT_LOAD_IGNORE_GLOBAL_ADVANCE_WIDTH);
     int max_width = FXFT_Get_Glyph_HoriAdvance(m_Face) * 1000 /
                     FXFT_Get_Face_UnitsPerEM(m_Face);
     if (max_width == min_width) {
+      FXFT_Free(m_Face, pMasters);
       return;
     }
     int param = min_param +