diff options
| author | Oliver Chang <ochang@chromium.org> | 2015-10-22 11:03:40 -0700 |
|---|---|---|
| committer | Oliver Chang <ochang@chromium.org> | 2015-10-22 11:03:40 -0700 |
| commit | de49cf173d12fe7a462168e2bf6c44971d7d28f8 (patch) | |
| tree | 369aa23abd878095ba28db0e8caaad0942112d9b | |
| parent | 162f227af9c5e28885a36e6ca06a09c12c6223e0 (diff) | |
| download | pdfium-de49cf173d12fe7a462168e2bf6c44971d7d28f8.tar.xz | |
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 .
| -rw-r--r-- | core/src/fxge/ge/fx_ge_text.cpp | 9 |
1 files 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 + |