summaryrefslogtreecommitdiff
path: root/core/fpdftext/cpdf_textpage.cpp
diff options
context:
space:
mode:
authorLei Zhang <thestig@chromium.org>2018-04-30 22:25:03 +0000
committerChromium commit bot <commit-bot@chromium.org>2018-04-30 22:25:03 +0000
commit8dbad1ef10c86c893581696a87be32d04105d3a3 (patch)
treec5961329df4aecbeb8d1efe8172b242581f15af6 /core/fpdftext/cpdf_textpage.cpp
parent3d3c2dea9fcb7372e195f453a5e4ec3fd47392b9 (diff)
downloadpdfium-8dbad1ef10c86c893581696a87be32d04105d3a3.tar.xz
Add CPDF_TextPage::GetPrevCharInfo() helper method.
Change-Id: Ie5bea82757682390b274ad2da77d1686cc597046 Reviewed-on: https://pdfium-review.googlesource.com/31657 Reviewed-by: Ryan Harrison <rharrison@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org>
Diffstat (limited to 'core/fpdftext/cpdf_textpage.cpp')
-rw-r--r--core/fpdftext/cpdf_textpage.cpp51
1 files changed, 23 insertions, 28 deletions
diff --git a/core/fpdftext/cpdf_textpage.cpp b/core/fpdftext/cpdf_textpage.cpp
index 00ae5ad4d6..81637624ee 100644
--- a/core/fpdftext/cpdf_textpage.cpp
+++ b/core/fpdftext/cpdf_textpage.cpp
@@ -887,14 +887,12 @@ void CPDF_TextPage::ProcessMarkedContent(PDFTEXT_Obj Obj) {
}
void CPDF_TextPage::FindPreviousTextObject() {
- if (m_TempCharList.empty() && m_CharList.empty())
+ const PAGECHAR_INFO* pPrevCharInfo = GetPrevCharInfo();
+ if (!pPrevCharInfo)
return;
- PAGECHAR_INFO preChar =
- m_TempCharList.empty() ? m_CharList.back() : m_TempCharList.back();
-
- if (preChar.m_pTextObj)
- m_pPreTextObj = preChar.m_pTextObj;
+ if (pPrevCharInfo->m_pTextObj)
+ m_pPreTextObj = pPrevCharInfo->m_pTextObj;
}
void CPDF_TextPage::SwapTempTextBuf(int32_t iCharListStartAppend,
@@ -1209,16 +1207,15 @@ bool CPDF_TextPage::IsHyphen(wchar_t curChar) const {
return true;
}
- const PAGECHAR_INFO* preInfo;
- if (!m_TempCharList.empty())
- preInfo = &m_TempCharList.back();
- else if (!m_CharList.empty())
- preInfo = &m_CharList.back();
- else
- return false;
+ const PAGECHAR_INFO* pPrevCharInfo = GetPrevCharInfo();
+ return pPrevCharInfo && pPrevCharInfo->m_Flag == FPDFTEXT_CHAR_PIECE &&
+ IsHyphenCode(pPrevCharInfo->m_Unicode);
+}
- return FPDFTEXT_CHAR_PIECE == preInfo->m_Flag &&
- IsHyphenCode(preInfo->m_Unicode);
+const PAGECHAR_INFO* CPDF_TextPage::GetPrevCharInfo() const {
+ if (!m_TempCharList.empty())
+ return &m_TempCharList.back();
+ return !m_CharList.empty() ? &m_CharList.back() : nullptr;
}
CPDF_TextPage::GenerateCharacter CPDF_TextPage::ProcessInsertObject(
@@ -1440,12 +1437,8 @@ bool CPDF_TextPage::IsSameAsPreTextObject(
}
Optional<PAGECHAR_INFO> CPDF_TextPage::GenerateCharInfo(wchar_t unicode) {
- const PAGECHAR_INFO* preChar;
- if (!m_TempCharList.empty())
- preChar = &m_TempCharList.back();
- else if (!m_CharList.empty())
- preChar = &m_CharList.back();
- else
+ const PAGECHAR_INFO* pPrevCharInfo = GetPrevCharInfo();
+ if (!pPrevCharInfo)
return {};
PAGECHAR_INFO info;
@@ -1456,18 +1449,20 @@ Optional<PAGECHAR_INFO> CPDF_TextPage::GenerateCharInfo(wchar_t unicode) {
info.m_Flag = FPDFTEXT_CHAR_GENERATED;
int preWidth = 0;
- if (preChar->m_pTextObj && preChar->m_CharCode != -1) {
- preWidth =
- GetCharWidth(preChar->m_CharCode, preChar->m_pTextObj->GetFont());
+ if (pPrevCharInfo->m_pTextObj && pPrevCharInfo->m_CharCode != -1) {
+ preWidth = GetCharWidth(pPrevCharInfo->m_CharCode,
+ pPrevCharInfo->m_pTextObj->GetFont());
}
- float fFontSize = preChar->m_pTextObj ? preChar->m_pTextObj->GetFontSize()
- : preChar->m_CharBox.Height();
+ float fFontSize = pPrevCharInfo->m_pTextObj
+ ? pPrevCharInfo->m_pTextObj->GetFontSize()
+ : pPrevCharInfo->m_CharBox.Height();
if (!fFontSize)
fFontSize = kDefaultFontSize;
- info.m_Origin = CFX_PointF(
- preChar->m_Origin.x + preWidth * (fFontSize) / 1000, preChar->m_Origin.y);
+ info.m_Origin =
+ CFX_PointF(pPrevCharInfo->m_Origin.x + preWidth * (fFontSize) / 1000,
+ pPrevCharInfo->m_Origin.y);
info.m_CharBox = CFX_FloatRect(info.m_Origin.x, info.m_Origin.y,
info.m_Origin.x, info.m_Origin.y);
return info;