commit | 4cb707994456065c975d82796118aacb5d81fef3 | [log] [tgz] |
---|---|---|
author | Lei Zhang <thestig@chromium.org> | Mon Aug 19 22:39:21 2019 +0000 |
committer | Chromium commit bot <commit-bot@chromium.org> | Mon Aug 19 22:39:21 2019 +0000 |
tree | 634a2ef9c322aa82e95de87e2e09ae02ed4793ba | |
parent | d41929adaddfae9547d4d8e1993345906d2ff272 [diff] |
Remove an impossible condition in CPDF_CMapParser::ParseWord(). The one and only caller to ParseWord() checks and handles the same condition. Replace it with an assertion. Change-Id: I98a2c6ec544dcbd5e07fd8ba2d0a0aa08001baa6 Reviewed-on: https://pdfium-review.googlesource.com/c/pdfium/+/59512 Commit-Queue: Lei Zhang <thestig@chromium.org> Reviewed-by: Tom Sepez <tsepez@chromium.org>
diff --git a/core/fpdfapi/font/cpdf_cmapparser.cpp b/core/fpdfapi/font/cpdf_cmapparser.cpp index a4a8c14..1936ee6 100644 --- a/core/fpdfapi/font/cpdf_cmapparser.cpp +++ b/core/fpdfapi/font/cpdf_cmapparser.cpp
@@ -31,8 +31,7 @@ CPDF_CMapParser::~CPDF_CMapParser() = default; void CPDF_CMapParser::ParseWord(ByteStringView word) { - if (word.IsEmpty()) - return; + ASSERT(!word.IsEmpty()); if (word == "begincidchar") { m_Status = kProcessingCidChar;