diff options
| author | Carl Hetherington <cth@carlh.net> | 2022-08-15 16:54:40 +0200 |
|---|---|---|
| committer | Carl Hetherington <cth@carlh.net> | 2022-08-15 16:54:40 +0200 |
| commit | fdf2d34c513dcf3ab36295e69e7e0c0766789ded (patch) | |
| tree | ced43a975749b21ec15d4fa5a2b1b7c18b6612f3 /src/verify.cc | |
| parent | f13d0a886e36977c13bb41f915e75ad9db8ba850 (diff) | |
Stop assuming that the presence of FullContentTitleText means that there was CPL metadata.
Diffstat (limited to 'src/verify.cc')
| -rw-r--r-- | src/verify.cc | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/src/verify.cc b/src/verify.cc index bef05a16..e698920c 100644 --- a/src/verify.cc +++ b/src/verify.cc @@ -1534,10 +1534,7 @@ dcp::verify ( notes.push_back ({VerificationNote::Type::BV21_ERROR, VerificationNote::Code::INVALID_CLOSED_CAPTION_LINE_LENGTH}); } - if (!cpl->full_content_title_text()) { - /* Since FullContentTitleText is assumed always to exist if there's a CompositionMetadataAsset we - * can use it as a proxy for CompositionMetadataAsset's existence. - */ + if (!cpl->read_composition_metadata()) { notes.push_back ({VerificationNote::Type::BV21_ERROR, VerificationNote::Code::MISSING_CPL_METADATA, cpl->id(), cpl->file().get()}); } else if (!cpl->version_number()) { notes.push_back ({VerificationNote::Type::BV21_ERROR, VerificationNote::Code::MISSING_CPL_METADATA_VERSION_NUMBER, cpl->id(), cpl->file().get()}); |