From 69e19e2c0b6753a69470840226456858cdef5773 Mon Sep 17 00:00:00 2001 From: Lei Zhang Date: Tue, 7 Aug 2018 23:24:09 +0000 Subject: Make CPDF_Font::GetFontBBox() return the bounding box. MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Instead of using an out parameter. Fix nits in the only caller as well. Change-Id: I04eb92705b2fbda001191c36da972eee499acbc7 Reviewed-on: https://pdfium-review.googlesource.com/39490 Reviewed-by: Nicolás Peña Moreno Commit-Queue: Lei Zhang --- core/fpdfapi/font/cpdf_font.h | 2 +- fpdfsdk/pwl/cpwl_edit.cpp | 5 ++--- fpdfsdk/pwl/cpwl_edit.h | 3 +-- 3 files changed, 4 insertions(+), 6 deletions(-) diff --git a/core/fpdfapi/font/cpdf_font.h b/core/fpdfapi/font/cpdf_font.h index d79b13fb6b..f246293c3e 100644 --- a/core/fpdfapi/font/cpdf_font.h +++ b/core/fpdfapi/font/cpdf_font.h @@ -72,7 +72,7 @@ class CPDF_Font { FXFT_Face GetFace() const { return m_Font.GetFace(); } void AppendChar(ByteString* str, uint32_t charcode) const; - void GetFontBBox(FX_RECT& rect) const { rect = m_FontBBox; } + const FX_RECT& GetFontBBox() const { return m_FontBBox; } int GetTypeAscent() const { return m_Ascent; } int GetTypeDescent() const { return m_Descent; } uint32_t GetStringWidth(const ByteStringView& pString); diff --git a/fpdfsdk/pwl/cpwl_edit.cpp b/fpdfsdk/pwl/cpwl_edit.cpp index 41f2517a0a..5e28a4aa02 100644 --- a/fpdfsdk/pwl/cpwl_edit.cpp +++ b/fpdfsdk/pwl/cpwl_edit.cpp @@ -369,14 +369,13 @@ bool CPWL_Edit::IsTextFull() const { return m_pEdit->IsTextFull(); } -float CPWL_Edit::GetCharArrayAutoFontSize(CPDF_Font* pFont, +float CPWL_Edit::GetCharArrayAutoFontSize(const CPDF_Font* pFont, const CFX_FloatRect& rcPlate, int32_t nCharArray) { if (!pFont || pFont->IsStandardFont()) return 0.0f; - FX_RECT rcBBox; - pFont->GetFontBBox(rcBBox); + const FX_RECT& rcBBox = pFont->GetFontBBox(); CFX_FloatRect rcCell = rcPlate; float xdiv = rcCell.Width() / nCharArray * 1000.0f / rcBBox.Width(); diff --git a/fpdfsdk/pwl/cpwl_edit.h b/fpdfsdk/pwl/cpwl_edit.h index 3f9cd433f5..ce1d682dad 100644 --- a/fpdfsdk/pwl/cpwl_edit.h +++ b/fpdfsdk/pwl/cpwl_edit.h @@ -84,7 +84,7 @@ class CPWL_Edit : public CPWL_EditCtrl { bool IsTextFull() const; - static float GetCharArrayAutoFontSize(CPDF_Font* pFont, + static float GetCharArrayAutoFontSize(const CPDF_Font* pFont, const CFX_FloatRect& rcPlate, int32_t nCharArray); @@ -114,7 +114,6 @@ class CPWL_Edit : public CPWL_EditCtrl { bool IsVScrollBarVisible() const; void SetParamByFlag(); - float GetCharArrayAutoFontSize(int32_t nCharArray); CFX_PointF GetWordRightBottomPoint(const CPVT_WordPlace& wpWord); CPVT_WordRange CombineWordRange(const CPVT_WordRange& wr1, -- cgit v1.2.3