summaryrefslogtreecommitdiff
path: root/core/fpdftext
diff options
context:
space:
mode:
authorLei Zhang <thestig@chromium.org>2018-05-01 18:47:54 +0000
committerChromium commit bot <commit-bot@chromium.org>2018-05-01 18:47:54 +0000
commit4bbbca4880fdff6b106bd935e6cd62f82025ec55 (patch)
treedfe88a24e9df312a99c1dcfa08d66a4118f98c22 /core/fpdftext
parent13339176b157eab56a1fa0a11525e193c3aa597a (diff)
downloadpdfium-4bbbca4880fdff6b106bd935e6cd62f82025ec55.tar.xz
Clean up CFX_BidiString.
- Refer to the string in CFX_BidiString by const-ref. - Remove useless CharAt() method. - Turn a member variable into a local variable. Change-Id: I30f221b7350150c839a793129789d8ea7cc1f331 Reviewed-on: https://pdfium-review.googlesource.com/31670 Reviewed-by: dsinclair <dsinclair@chromium.org> Commit-Queue: Lei Zhang <thestig@chromium.org>
Diffstat (limited to 'core/fpdftext')
-rw-r--r--core/fpdftext/cpdf_textpage.cpp4
1 files changed, 2 insertions, 2 deletions
diff --git a/core/fpdftext/cpdf_textpage.cpp b/core/fpdftext/cpdf_textpage.cpp
index 31ed9fc704..cc0c9db8a1 100644
--- a/core/fpdftext/cpdf_textpage.cpp
+++ b/core/fpdftext/cpdf_textpage.cpp
@@ -702,11 +702,11 @@ void CPDF_TextPage::CloseTempLine() {
eCurrentDirection == CFX_BidiChar::RIGHT)) {
eCurrentDirection = CFX_BidiChar::RIGHT;
for (int m = segment.start + segment.count; m > segment.start; --m)
- AddCharInfoByRLDirection(bidi.CharAt(m - 1), m_TempCharList[m - 1]);
+ AddCharInfoByRLDirection(str[m - 1], m_TempCharList[m - 1]);
} else {
eCurrentDirection = CFX_BidiChar::LEFT;
for (int m = segment.start; m < segment.start + segment.count; m++)
- AddCharInfoByLRDirection(bidi.CharAt(m), m_TempCharList[m]);
+ AddCharInfoByLRDirection(str[m], m_TempCharList[m]);
}
}
m_TempCharList.clear();