diff options
| author | Tom Sepez <tsepez@chromium.org> | 2016-03-18 09:32:06 -0700 |
|---|---|---|
| committer | Tom Sepez <tsepez@chromium.org> | 2016-03-18 09:32:06 -0700 |
| commit | 314743a4a752716b8e977482ad6c757c70d019db (patch) | |
| tree | 3bb2332e5232c1b37e9da0ccb02aec2e87de1a00 /xfa/fxbarcode/pdf417/BC_PDF417.h | |
| parent | 3420909bbb47d6e47d6c561cbcce06d056fdf0a3 (diff) | |
| download | pdfium-314743a4a752716b8e977482ad6c757c70d019db.tar.xz | |
Reduce size of several PDFium constant tables.
Either by using the smallest possible data type, or
By accurately calcuating the bounds, or
Moving to array of pointers for varying length rows.
Notes:
The 929 is reassuring since PDF417 barcode uses GF(929).
FX_WCHAR is 4 bytes on unix (2 on windows).
Binary looks to be about 240k smaller on linux/64
R=ochang@chromium.org
Review URL: https://codereview.chromium.org/1807373002 .
Diffstat (limited to 'xfa/fxbarcode/pdf417/BC_PDF417.h')
| -rw-r--r-- | xfa/fxbarcode/pdf417/BC_PDF417.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/xfa/fxbarcode/pdf417/BC_PDF417.h b/xfa/fxbarcode/pdf417/BC_PDF417.h index fe279db21b..1aec33a964 100644 --- a/xfa/fxbarcode/pdf417/BC_PDF417.h +++ b/xfa/fxbarcode/pdf417/BC_PDF417.h @@ -30,9 +30,9 @@ class CBC_PDF417 { void setCompact(FX_BOOL compact); private: - static int32_t START_PATTERN; - static int32_t STOP_PATTERN; - static int32_t CODEWORD_TABLE[][1000]; + static const int32_t START_PATTERN = 0x1fea8; + static const int32_t STOP_PATTERN = 0x3fa29; + static const int32_t CODEWORD_TABLE[][929]; static FX_FLOAT PREFERRED_RATIO; static FX_FLOAT DEFAULT_MODULE_WIDTH; static FX_FLOAT HEIGHT; |