From: Carl Hetherington <cth@carlh.net>
Date: Sun, 7 Jan 2024 23:35:29 +0000 (+0100)
Subject: Add OK note when CPL has valid annotation text.
X-Git-Url: https://git.carlh.net/gitweb/?a=commitdiff_plain;ds=inline;h=342a1a8b45cb408ec1579ceb7f7eae9367bf6a1b;p=libdcp.git

Add OK note when CPL has valid annotation text.
---

diff --git a/src/verify.cc b/src/verify.cc
index 7aa28b43..a59ce82b 100644
--- a/src/verify.cc
+++ b/src/verify.cc
@@ -1603,6 +1603,8 @@ verify_cpl(Context& context, shared_ptr<const CPL> cpl)
 			context.bv21_error(VerificationNote::Code::MISSING_CPL_ANNOTATION_TEXT, cpl->file().get());
 		} else if (cpl->annotation_text().get() != cpl->content_title_text()) {
 			context.warning(VerificationNote::Code::MISMATCHED_CPL_ANNOTATION_TEXT, cpl->file().get());
+		} else {
+			context.ok(VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT, cpl->annotation_text().get());
 		}
 	}
 
@@ -2047,6 +2049,8 @@ dcp::note_to_string(VerificationNote note, function<string (string)> process_str
 		return compose("The CPL %1 has no <AnnotationText> tag.", note.cpl_id().get());
 	case VerificationNote::Code::MISMATCHED_CPL_ANNOTATION_TEXT:
 		return compose("The CPL %1 has an <AnnotationText> which differs from its <ContentTitleText>.", note.cpl_id().get());
+	case VerificationNote::Code::VALID_CPL_ANNOTATION_TEXT:
+		return compose("Valid CPL annotation text %1", note.note().get());
 	case VerificationNote::Code::MISMATCHED_ASSET_DURATION:
 		return process_string("All assets in a reel do not have the same duration.");
 	case VerificationNote::Code::MISSING_MAIN_SUBTITLE_FROM_SOME_REELS:
diff --git a/src/verify.h b/src/verify.h
index 24ada64d..b0cf0b4a 100644
--- a/src/verify.h
+++ b/src/verify.h
@@ -265,6 +265,7 @@ public:
 		 *  file contains the CPL filename
 		 */
 		MISMATCHED_CPL_ANNOTATION_TEXT,
+		VALID_CPL_ANNOTATION_TEXT,
 		/** At least one asset in a reel does not have the same duration as the others */
 		MISMATCHED_ASSET_DURATION,
 		/** If one reel has a _MainSubtitle_, all must have them */