Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(45)

Side by Side Diff: core/src/fpdfapi/fpdf_parser/fpdf_parser_decode.cpp

Issue 1431683008: Revert "Revert "Revert "Cleanup some numeric code.""" (Closed) Base URL: https://pdfium.googlesource.com/pdfium.git@master
Patch Set: Created 5 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2014 PDFium Authors. All rights reserved. 1 // Copyright 2014 PDFium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com 5 // Original code copyright 2014 Foxit Software Inc. http://www.foxitsoftware.com
6 6
7 #include <limits.h> 7 #include <limits.h>
8 #include "../../../include/fpdfapi/fpdf_parser.h" 8 #include "../../../include/fpdfapi/fpdf_parser.h"
9 #include "../../../include/fpdfapi/fpdf_module.h" 9 #include "../../../include/fpdfapi/fpdf_module.h"
10 #include "../../../include/fxcodec/fx_codec.h" 10 #include "../../../include/fxcodec/fx_codec.h"
11 #include "../../../include/fxcrt/fx_ext.h"
12 11
13 #define _STREAM_MAX_SIZE_ 20 * 1024 * 1024 12 #define _STREAM_MAX_SIZE_ 20 * 1024 * 1024
14 13
15 const FX_WORD PDFDocEncoding[256] = { 14 const FX_WORD PDFDocEncoding[256] = {
16 0x0000, 0x0001, 0x0002, 0x0003, 0x0004, 0x0005, 0x0006, 0x0007, 0x0008, 15 0x0000, 0x0001, 0x0002, 0x0003, 0x0004, 0x0005, 0x0006, 0x0007, 0x0008,
17 0x0009, 0x000a, 0x000b, 0x000c, 0x000d, 0x000e, 0x000f, 0x0010, 0x0011, 16 0x0009, 0x000a, 0x000b, 0x000c, 0x000d, 0x000e, 0x000f, 0x0010, 0x0011,
18 0x0012, 0x0013, 0x0014, 0x0015, 0x0016, 0x0017, 0x02d8, 0x02c7, 0x02c6, 17 0x0012, 0x0013, 0x0014, 0x0015, 0x0016, 0x0017, 0x02d8, 0x02c7, 0x02c6,
19 0x02d9, 0x02dd, 0x02db, 0x02da, 0x02dc, 0x0020, 0x0021, 0x0022, 0x0023, 18 0x02d9, 0x02dd, 0x02db, 0x02da, 0x02dc, 0x0020, 0x0021, 0x0022, 0x0023,
20 0x0024, 0x0025, 0x0026, 0x0027, 0x0028, 0x0029, 0x002a, 0x002b, 0x002c, 19 0x0024, 0x0025, 0x0026, 0x0027, 0x0028, 0x0029, 0x002a, 0x002b, 0x002c,
21 0x002d, 0x002e, 0x002f, 0x0030, 0x0031, 0x0032, 0x0033, 0x0034, 0x0035, 20 0x002d, 0x002e, 0x002f, 0x0030, 0x0031, 0x0032, 0x0033, 0x0034, 0x0035,
(...skipping 100 matching lines...) Expand 10 before | Expand all | Expand 10 after
122 FX_DWORD src_size, 121 FX_DWORD src_size,
123 uint8_t*& dest_buf, 122 uint8_t*& dest_buf,
124 FX_DWORD& dest_size) { 123 FX_DWORD& dest_size) {
125 FX_DWORD i; 124 FX_DWORD i;
126 for (i = 0; i < src_size; i++) 125 for (i = 0; i < src_size; i++)
127 if (src_buf[i] == '>') { 126 if (src_buf[i] == '>') {
128 break; 127 break;
129 } 128 }
130 dest_buf = FX_Alloc(uint8_t, i / 2 + 1); 129 dest_buf = FX_Alloc(uint8_t, i / 2 + 1);
131 dest_size = 0; 130 dest_size = 0;
132 bool bFirst = true; 131 FX_BOOL bFirstDigit = TRUE;
133 for (i = 0; i < src_size; i++) { 132 for (i = 0; i < src_size; i++) {
134 uint8_t ch = src_buf[i]; 133 uint8_t ch = src_buf[i];
135 if (PDFCharIsLineEnding(ch) || ch == ' ' || ch == '\t') 134 if (PDFCharIsLineEnding(ch) || ch == ' ' || ch == '\t')
136 continue; 135 continue;
137 136
138 if (ch == '>') { 137 int digit;
139 ++i; 138 if (ch <= '9' && ch >= '0') {
139 digit = ch - '0';
140 } else if (ch <= 'f' && ch >= 'a') {
141 digit = ch - 'a' + 10;
142 } else if (ch <= 'F' && ch >= 'A') {
143 digit = ch - 'A' + 10;
144 } else if (ch == '>') {
145 i++;
140 break; 146 break;
147 } else {
148 continue;
141 } 149 }
142 if (!std::isxdigit(ch)) 150 if (bFirstDigit) {
143 continue;
144
145 int digit = FXSYS_toHexDigit(ch);
146 if (bFirst)
147 dest_buf[dest_size] = digit * 16; 151 dest_buf[dest_size] = digit * 16;
148 else 152 } else {
149 dest_buf[dest_size++] += digit; 153 dest_buf[dest_size++] += digit;
150 154 }
151 bFirst = !bFirst; 155 bFirstDigit = !bFirstDigit;
152 } 156 }
153 if (!bFirst) 157 if (!bFirstDigit) {
154 dest_size++; 158 dest_size++;
159 }
155 return i; 160 return i;
156 } 161 }
157
158 FX_DWORD RunLengthDecode(const uint8_t* src_buf, 162 FX_DWORD RunLengthDecode(const uint8_t* src_buf,
159 FX_DWORD src_size, 163 FX_DWORD src_size,
160 uint8_t*& dest_buf, 164 uint8_t*& dest_buf,
161 FX_DWORD& dest_size) { 165 FX_DWORD& dest_size) {
162 FX_DWORD i = 0; 166 FX_DWORD i = 0;
163 FX_DWORD old; 167 FX_DWORD old;
164 dest_size = 0; 168 dest_size = 0;
165 while (i < src_size) { 169 while (i < src_size) {
166 if (src_buf[i] < 128) { 170 if (src_buf[i] < 128) {
167 old = dest_size; 171 old = dest_size;
(...skipping 401 matching lines...) Expand 10 before | Expand all | Expand 10 after
569 FX_DWORD src_size, 573 FX_DWORD src_size,
570 uint8_t*& dest_buf, 574 uint8_t*& dest_buf,
571 FX_DWORD& dest_size) { 575 FX_DWORD& dest_size) {
572 CCodec_ModuleMgr* pEncoders = CPDF_ModuleMgr::Get()->GetCodecModule(); 576 CCodec_ModuleMgr* pEncoders = CPDF_ModuleMgr::Get()->GetCodecModule();
573 if (pEncoders) { 577 if (pEncoders) {
574 return pEncoders->GetFlateModule()->FlateOrLZWDecode( 578 return pEncoders->GetFlateModule()->FlateOrLZWDecode(
575 FALSE, src_buf, src_size, FALSE, 0, 0, 0, 0, 0, dest_buf, dest_size); 579 FALSE, src_buf, src_size, FALSE, 0, 0, 0, 0, 0, dest_buf, dest_size);
576 } 580 }
577 return 0; 581 return 0;
578 } 582 }
OLDNEW
« no previous file with comments | « core/src/fpdfapi/fpdf_page/fpdf_page_parser_old.cpp ('k') | core/src/fpdfapi/fpdf_parser/fpdf_parser_parser.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698