diff options
author | Lei Zhang <thestig@chromium.org> | 2018-05-01 18:47:54 +0000 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2018-05-01 18:47:54 +0000 |
commit | 4bbbca4880fdff6b106bd935e6cd62f82025ec55 (patch) | |
tree | dfe88a24e9df312a99c1dcfa08d66a4118f98c22 /core/fpdftext/cpdf_textpage.cpp | |
parent | 13339176b157eab56a1fa0a11525e193c3aa597a (diff) | |
download | pdfium-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/cpdf_textpage.cpp')
-rw-r--r-- | core/fpdftext/cpdf_textpage.cpp | 4 |
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(); |