diff options
| author | Carl Hetherington <cth@carlh.net> | 2025-03-25 21:34:22 +0100 |
|---|---|---|
| committer | Carl Hetherington <cth@carlh.net> | 2025-03-25 21:34:22 +0100 |
| commit | bc29132b361026a32bfe29418d1aa3b783d2e7a6 (patch) | |
| tree | c226c06404b868796440edda702234de4e03546b /src | |
| parent | 7abe0aaa61fb98faf97c595ffaae86f3794217c6 (diff) | |
Don't give an invalid picture size error for MPEG2 picture assets.
Diffstat (limited to 'src')
| -rw-r--r-- | src/verify.cc | 20 |
1 files changed, 13 insertions, 7 deletions
diff --git a/src/verify.cc b/src/verify.cc index 458883d5..b6df354c 100644 --- a/src/verify.cc +++ b/src/verify.cc @@ -562,13 +562,19 @@ verify_main_picture_asset(Context& context, shared_ptr<const ReelPictureAsset> r verify_picture_details(context, reel_asset, file, start_frame); } - /* Only flat/scope allowed by Bv2.1 */ - if ( - asset->size() != Size(2048, 858) && - asset->size() != Size(1998, 1080) && - asset->size() != Size(4096, 1716) && - asset->size() != Size(3996, 2160)) { - context.bv21_error(VerificationNote::Code::INVALID_PICTURE_SIZE_IN_PIXELS, String::compose("%1x%2", asset->size().width, asset->size().height), file); + if (dynamic_pointer_cast<const J2KPictureAsset>(asset)) { + /* Only flat/scope allowed by Bv2.1 */ + if ( + asset->size() != Size(2048, 858) && + asset->size() != Size(1998, 1080) && + asset->size() != Size(4096, 1716) && + asset->size() != Size(3996, 2160)) { + context.bv21_error(VerificationNote::Code::INVALID_PICTURE_SIZE_IN_PIXELS, String::compose("%1x%2", asset->size().width, asset->size().height), file); + } + } else if (dynamic_pointer_cast<const MPEG2PictureAsset>(asset)) { + if (asset->size() != Size(1920, 1080)) { + context.error(VerificationNote::Code::INVALID_PICTURE_SIZE_IN_PIXELS, fmt::format("{}x{}", asset->size().width, asset->size().height), file); + } } /* Only 24, 25, 48fps allowed for 2K */ |
