From de49cf173d12fe7a462168e2bf6c44971d7d28f8 Mon Sep 17 00:00:00 2001 From: Oliver Chang Date: Thu, 22 Oct 2015 11:03:40 -0700 Subject: Remove unused error variable from CFX_Font::AdjustMMParams 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 | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/core/src/fxge/ge/fx_ge_text.cpp b/core/src/fxge/ge/fx_ge_text.cpp index 11bc29ef5b..6cc9790e56 100644 --- a/core/src/fxge/ge/fx_ge_text.cpp +++ b/core/src/fxge/ge/fx_ge_text.cpp @@ -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 + -- cgit v1.2.3