diff options
author | Lei Zhang <thestig@chromium.org> | 2018-08-09 08:05:04 +0000 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2018-08-09 08:05:04 +0000 |
commit | 60627d6eafd025dde711e532eee6866840c04bef (patch) | |
tree | 0988f5945d243edb5eea2e189d8e52c180ffba0d /core/fpdfapi/font/cpdf_cmap.cpp | |
parent | 74557b1b4746bc80283f0acf248242fdcab71deb (diff) | |
download | pdfium-60627d6eafd025dde711e532eee6866840c04bef.tar.xz |
Change CPDF_Font::GetNextChar()'s in-out parameter to pass by pointer.chromium/3518
Instead of passing by non-const reference. Adjust the override in
CPDF_CIDFont and CPDF_CMap::GetNextChar() as well.
Change-Id: I0ee6dc21077101cbeeb0e334498075fc463a2d65
Reviewed-on: https://pdfium-review.googlesource.com/39492
Reviewed-by: Nicolás Peña Moreno <npm@chromium.org>
Commit-Queue: Lei Zhang <thestig@chromium.org>
Diffstat (limited to 'core/fpdfapi/font/cpdf_cmap.cpp')
-rw-r--r-- | core/fpdfapi/font/cpdf_cmap.cpp | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/core/fpdfapi/font/cpdf_cmap.cpp b/core/fpdfapi/font/cpdf_cmap.cpp index f3ef829e20..ee8f264869 100644 --- a/core/fpdfapi/font/cpdf_cmap.cpp +++ b/core/fpdfapi/font/cpdf_cmap.cpp @@ -338,7 +338,8 @@ uint16_t CPDF_CMap::CIDFromCharCode(uint32_t charcode) const { } uint32_t CPDF_CMap::GetNextChar(const ByteStringView& pString, - size_t& offset) const { + size_t* pOffset) const { + size_t& offset = *pOffset; auto pBytes = pString.span(); switch (m_CodingScheme) { case OneByte: { @@ -422,7 +423,7 @@ size_t CPDF_CMap::CountChar(const ByteStringView& pString) const { size_t count = 0; size_t offset = 0; while (offset < pString.GetLength()) { - GetNextChar(pString, offset); + GetNextChar(pString, &offset); count++; } return count; |