diff options
| author | Lei Zhang <thestig@chromium.org> | 2018-04-12 15:37:39 +0000 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2018-04-12 15:37:39 +0000 |
| commit | 1c23a6d78c95ff0714cda6f642420e0502edac29 (patch) | |
| tree | 82d35256f9a214cc26f03d7c5626653a2398f56b /xfa/fgas/font | |
| parent | e6d4559a1b6c2eb0eb433b9db70745e3be3d8465 (diff) | |
| download | pdfium-1c23a6d78c95ff0714cda6f642420e0502edac29.tar.xz | |
Make CFX_Font take FX_RECT out params by pointer.
Change-Id: I6643aaf66bf3e9d53d5afab5c99a5cb7deab3661
Reviewed-on: https://pdfium-review.googlesource.com/30054
Reviewed-by: dsinclair <dsinclair@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Diffstat (limited to 'xfa/fgas/font')
| -rw-r--r-- | xfa/fgas/font/cfgas_gefont.cpp | 10 |
1 files changed, 2 insertions, 8 deletions
diff --git a/xfa/fgas/font/cfgas_gefont.cpp b/xfa/fgas/font/cfgas_gefont.cpp index 6b2cb0f03b..35d3480d03 100644 --- a/xfa/fgas/font/cfgas_gefont.cpp +++ b/xfa/fgas/font/cfgas_gefont.cpp @@ -202,7 +202,7 @@ bool CFGAS_GEFont::GetCharBBox(wchar_t wUnicode, FX_RECT* bbox) { return pFont->GetCharBBox(wUnicode, bbox); FX_RECT rtBBox; - if (!m_pFont->GetGlyphBBox(iGlyph, rtBBox)) + if (!m_pFont->GetGlyphBBox(iGlyph, &rtBBox)) return false; m_BBoxMap[wUnicode] = rtBBox; @@ -211,13 +211,7 @@ bool CFGAS_GEFont::GetCharBBox(wchar_t wUnicode, FX_RECT* bbox) { } bool CFGAS_GEFont::GetBBox(FX_RECT* bbox) { - // TODO(thestig): Pass directly into GetBBox(). - FX_RECT rt; - if (!m_pFont->GetBBox(rt)) - return false; - - *bbox = rt; - return true; + return m_pFont->GetBBox(bbox); } int32_t CFGAS_GEFont::GetGlyphIndex(wchar_t wUnicode) { |