diff options
| author | Carl Hetherington <cth@carlh.net> | 2021-11-22 20:57:07 +0100 |
|---|---|---|
| committer | Carl Hetherington <cth@carlh.net> | 2021-11-22 22:07:59 +0100 |
| commit | 9c36e773ca56bb8b6b5082b22dff20412901049d (patch) | |
| tree | 9eb59a34d588baeef9bb96479a716cf7e039d98c /src | |
| parent | 59e22e6d9ff4fc180f79df64da2a27f44a982b51 (diff) | |
Clarify some errors and add comments wondering about their validity.
Diffstat (limited to 'src')
| -rw-r--r-- | src/verify_j2k.cc | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/verify_j2k.cc b/src/verify_j2k.cc index 48d85a33..7193f64d 100644 --- a/src/verify_j2k.cc +++ b/src/verify_j2k.cc @@ -221,7 +221,8 @@ dcp::verify_j2k (shared_ptr<const Data> j2k, vector<VerificationNote>& notes) } else if (*marker_name == "COD") { num_COD++; get_16(); // length - require_8(1, "invalid coding style %1"); + /* XXX: I can't find any evidence for this: must the coding style really always be 1? */ + require_8(1, "invalid COD coding style %1"); require_8(4, "invalid progression order %1"); // CPRL require_16(1, "invalid quality layers count %1"); require_8(1, "invalid multi-component transform flag %1"); @@ -260,7 +261,8 @@ dcp::verify_j2k (shared_ptr<const Data> j2k, vector<VerificationNote>& notes) } else if (*marker_name == "COC") { get_16(); // length require_8(0, "invalid COC component number"); - require_8(1, "invalid coding style %1"); + /* XXX: I can't find any evidence for this: must the coding style really always be 1? */ + require_8(1, "invalid COC coding style %1"); require_8(5, "invalid number of transform levels %1"); require_8(3, "invalid code block width exponent %1"); require_8(3, "invalid code block height exponent %1"); |