diff options
author | Lei Zhang <thestig@chromium.org> | 2017-06-09 11:04:41 -0700 |
---|---|---|
committer | Chromium commit bot <commit-bot@chromium.org> | 2017-06-09 18:33:56 +0000 |
commit | 59c1ac070d1c8ca70e96805b697c5635ccd70cdf (patch) | |
tree | 52ea762d0c694e241ed05f071e4295a47eb29007 /core/fpdfapi/parser/fpdf_parser_decode.cpp | |
parent | 7e2d1f4cb2af916e0d5eb01bb6b9cb3ffa0736c3 (diff) | |
download | pdfium-59c1ac070d1c8ca70e96805b697c5635ccd70cdf.tar.xz |
Simplify CPDF_Array::RemoveAt(index, size).
Instead of one general RemoveAt() method, split it into:
- RemoveAt(index)
- Truncate(nNewSize)
- Clear()
Update callers, which are now easier to understand.
Update existing unit tests and add new tests.
Change-Id: I38fe40146ce8f2479677b2caadd20a1756678768
Reviewed-on: https://pdfium-review.googlesource.com/6417
Commit-Queue: Lei Zhang <thestig@chromium.org>
Reviewed-by: Nicolás Peña <npm@chromium.org>
Diffstat (limited to 'core/fpdfapi/parser/fpdf_parser_decode.cpp')
-rw-r--r-- | core/fpdfapi/parser/fpdf_parser_decode.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/core/fpdfapi/parser/fpdf_parser_decode.cpp b/core/fpdfapi/parser/fpdf_parser_decode.cpp index 561b6627bc..3d7b9ffd59 100644 --- a/core/fpdfapi/parser/fpdf_parser_decode.cpp +++ b/core/fpdfapi/parser/fpdf_parser_decode.cpp @@ -400,7 +400,7 @@ bool PDF_DataDecode(const uint8_t* src_buf, dest_buf = last_buf; dest_size = last_size; if (CPDF_Array* pDecoders = pDecoder->AsArray()) - pDecoders->RemoveAt(i + 1, pDecoders->GetCount() - i - 1); + pDecoders->Truncate(i + 1); return true; } if (last_buf != src_buf) { |