diff options
| author | Tom Sepez <tsepez@chromium.org> | 2015-03-19 12:51:04 -0700 |
|---|---|---|
| committer | Tom Sepez <tsepez@chromium.org> | 2015-03-19 12:51:04 -0700 |
| commit | d1f792ac1ba25494b44e664d4a427127299d05e6 (patch) | |
| tree | 0cdfa5e28c1c7df416b1cd531f9a4450668f5a91 /BUILD.gn | |
| parent | 74b81d9ff8b4c4c5306b8c4611ee780ea0628c52 (diff) | |
| download | pdfium-d1f792ac1ba25494b44e664d4a427127299d05e6.tar.xz | |
Fix subtle issues in opj_skip_from_memory and add unit tests.
Follow on to https://codereview.chromium.org/990683002/.
This more closely mimics what fseek() actually does, so as
to avoid subtle bugs down the road.
Move the DecodeData struct into a header so the test can
use it, and provide a constructor for it.
Along the way, I added include guards, removed the p_
prefix from some non-pointer vars, fixed some IWYU, and
resolved some signed/unsigned comparison warnings with
careful casting.
BUG=452671
R=jun_fang@foxitsoftware.com, thestig@chromium.org
Review URL: https://codereview.chromium.org/1016203002
Diffstat (limited to 'BUILD.gn')
| -rw-r--r-- | BUILD.gn | 1 |
1 files changed, 1 insertions, 0 deletions
@@ -846,6 +846,7 @@ static_library("formfiller") { test("pdfium_unittests") { sources = [ "core/src/fpdfapi/fpdf_parser/fpdf_parser_decode_unittest.cpp", + "core/src/fxcodec/codec/fx_codec_jpx_unittest.cpp", "core/src/fxcrt/fx_basic_bstring_unittest.cpp", "core/src/fxcrt/fx_basic_wstring_unittest.cpp", "testing/fx_string_testhelpers.cpp", |