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

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

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

core/src/fxge/ge/fx_ge_text.cpp

index 11bc29e..6cc9790 100644 (file)
@@ -1401,20 +1401,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 +