diff options
| author | Carl Hetherington <cth@carlh.net> | 2024-02-12 20:53:42 +0100 |
|---|---|---|
| committer | Carl Hetherington <cth@carlh.net> | 2024-02-12 20:53:42 +0100 |
| commit | 3a71f7ffe55658f6662fcdcd5c413bfad110a30e (patch) | |
| tree | 86f18b8f07171c5ffcaca2f28e2c4d39a2d18dea | |
| parent | eb04ac87ccfa046dd342ca7b9e6478c3bdcabbba (diff) | |
Fix message for high bitrate frames during verify.
| -rw-r--r-- | src/wx/verify_dcp_dialog.cc | 13 |
1 files changed, 9 insertions, 4 deletions
diff --git a/src/wx/verify_dcp_dialog.cc b/src/wx/verify_dcp_dialog.cc index e86cb2c80..2b108c945 100644 --- a/src/wx/verify_dcp_dialog.cc +++ b/src/wx/verify_dcp_dialog.cc @@ -94,6 +94,14 @@ VerifyDCPDialog::VerifyDCPDialog (wxWindow* parent, shared_ptr<VerifyDCPJob> job if (note.calculated_hash()) { message.Replace("%calculated_hash", std_to_wx(note.calculated_hash().get())); } + if (note.frame()) { + message.Replace("%frame", std_to_wx(dcp::raw_convert<string>(note.frame().get()))); + message.Replace( + "%timecode", + std_to_wx( + dcp::Time(note.frame().get(), note.frame_rate().get(), note.frame_rate().get()).as_string(dcp::Standard::SMPTE) + )); + } if (note.note()) { message.Replace("%n", std_to_wx(note.note().get())); } @@ -103,9 +111,6 @@ VerifyDCPDialog::VerifyDCPDialog (wxWindow* parent, shared_ptr<VerifyDCPJob> job if (note.line()) { message.Replace("%l", std_to_wx(dcp::raw_convert<string>(note.line().get()))); } - if (note.frame()) { - message.Replace("%frame", std_to_wx(dcp::raw_convert<string>(note.frame().get()))); - } if (note.component()) { message.Replace("%component", std_to_wx(dcp::raw_convert<string>(note.component().get()))); } @@ -179,7 +184,7 @@ VerifyDCPDialog::VerifyDCPDialog (wxWindow* parent, shared_ptr<VerifyDCPJob> job add(i, _("At least one frame of the video asset %f is over the limit of 250Mbit/s.")); break; case dcp::VerificationNote::Code::NEARLY_INVALID_PICTURE_FRAME_SIZE_IN_BYTES: - add(i, _("At least one frame of the video asset %f is close to the limit of 250MBit/s.")); + add(i, _("Frame %frame (timecode %timecode) in asset %f has an instantaneous bit rate that is close to the limit of 250Mbit/s.")); break; case dcp::VerificationNote::Code::EXTERNAL_ASSET: add(i, _("This DCP refers to at the asset %n in another DCP (and perhaps others), so it is a \"version file\" (VF)")); |