summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorCarl Hetherington <cth@carlh.net>2026-03-29 15:43:53 +0200
committerCarl Hetherington <cth@carlh.net>2026-04-01 23:43:00 +0200
commitb65880e549432818f750675a4f518eb96d1f486e (patch)
tree22d6f967e88d44b9b4e1f40e406798828223218c /src
parent814abc3a7afca78c8877d2c9478364c327db7734 (diff)
INVALID_PICTURE_SIZE_IN_PIXELS: note -> size_in_pixels.
Diffstat (limited to 'src')
-rw-r--r--src/verify.cc6
-rw-r--r--src/verify.h2
2 files changed, 4 insertions, 4 deletions
diff --git a/src/verify.cc b/src/verify.cc
index 41ecda0b..30b3ce43 100644
--- a/src/verify.cc
+++ b/src/verify.cc
@@ -574,11 +574,11 @@ verify_main_picture_asset(Context& context, shared_ptr<const ReelPictureAsset> r
asset->size() != Size(1998, 1080) &&
asset->size() != Size(4096, 1716) &&
asset->size() != Size(3996, 2160)) {
- context.add_note(VerificationNote::Code::INVALID_PICTURE_SIZE_IN_PIXELS, String::compose("%1x%2", asset->size().width, asset->size().height), file);
+ context.add_note(VerificationNote(VerificationNote::Code::INVALID_PICTURE_SIZE_IN_PIXELS, file).set_size_in_pixels(asset->size()));
}
} else if (dynamic_pointer_cast<const MPEG2PictureAsset>(asset)) {
if (asset->size() != Size(1920, 1080)) {
- context.add_note(VerificationNote::Code::INVALID_PICTURE_SIZE_IN_PIXELS, fmt::format("{}x{}", asset->size().width, asset->size().height), file);
+ context.add_note(VerificationNote(VerificationNote::Code::INVALID_PICTURE_SIZE_IN_PIXELS, file).set_size_in_pixels(asset->size()));
}
}
@@ -2016,7 +2016,7 @@ dcp::note_to_string(VerificationNote note, function<string (string)> process_str
case VerificationNote::Code::VALID_RELEASE_TERRITORY:
return compose("Valid release territory %1.", *note.territory());
case VerificationNote::Code::INVALID_PICTURE_SIZE_IN_PIXELS:
- return compose("The size %1 of picture asset %2 is not allowed.", note.note().get(), filename());
+ return compose("The size %1x%2 of picture asset %3 is not allowed.", note.size_in_pixels()->width, note.size_in_pixels()->height, filename());
case VerificationNote::Code::INVALID_PICTURE_FRAME_RATE_FOR_2K:
return compose("The frame rate %1/%2 of picture asset %3 is not allowed for 2K DCPs.", note.frame_rate()->numerator, note.frame_rate()->denominator, filename());
case VerificationNote::Code::INVALID_PICTURE_FRAME_RATE_FOR_4K:
diff --git a/src/verify.h b/src/verify.h
index ecaeacfe..cab264db 100644
--- a/src/verify.h
+++ b/src/verify.h
@@ -209,7 +209,7 @@ public:
*/
VALID_RELEASE_TERRITORY,
/** A picture asset does not have one of the required Bv2.1 sizes (in pixels) [Bv2.1_7.1]
- * note contains the incorrect size as "<width>x<height>"
+ * size_in_pixels contains the incorrect size
* file contains the asset filename
* reel_index contains the reel index (starting from 0)
*/