diff options
| author | Tom Sepez <tsepez@chromium.org> | 2017-05-04 15:32:21 -0700 |
|---|---|---|
| committer | Chromium commit bot <commit-bot@chromium.org> | 2017-05-05 16:20:01 +0000 |
| commit | 4c64450875263a1f48e84f6d2223d8c36484f7cf (patch) | |
| tree | 4f1b8227583713df726612a368cf5449bfe323db /core/fpdfapi/font/fpdf_font_cid_unittest.cpp | |
| parent | e8c1d4144e8407c0631116a954fa347dd39375ff (diff) | |
| download | pdfium-4c64450875263a1f48e84f6d2223d8c36484f7cf.tar.xz | |
Fix CPDF_CMap::m_pAddMapping lack of type information.
Using a ByteBuffer and an array of uint8_t's isn't how one
would represent an array of structured data. Packing uint16_t's
into a uint32_t via / and % isn't ideal, either.
Bug:
Change-Id: Ib09ae2659ba2f027724546bb7aef99bdfd2dea25
Reviewed-on: https://pdfium-review.googlesource.com/4951
Commit-Queue: Tom Sepez <tsepez@chromium.org>
Reviewed-by: Lei Zhang <thestig@chromium.org>
Diffstat (limited to 'core/fpdfapi/font/fpdf_font_cid_unittest.cpp')
| -rw-r--r-- | core/fpdfapi/font/fpdf_font_cid_unittest.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/core/fpdfapi/font/fpdf_font_cid_unittest.cpp b/core/fpdfapi/font/fpdf_font_cid_unittest.cpp index 53f5e47f92..813a3a9d9b 100644 --- a/core/fpdfapi/font/fpdf_font_cid_unittest.cpp +++ b/core/fpdfapi/font/fpdf_font_cid_unittest.cpp @@ -36,7 +36,7 @@ TEST(fpdf_font_cid, CMap_GetCode) { } TEST(fpdf_font_cid, CMap_GetCodeRange) { - CMap_CodeRange range; + CPDF_CMap::CodeRange range; // Must start with a < EXPECT_FALSE(CPDF_CMapParser::CMap_GetCodeRange(range, "", "")); |