Disable some hints when encoding to MPEG2.
authorCarl Hetherington <cth@carlh.net>
Sun, 7 Apr 2024 18:12:20 +0000 (20:12 +0200)
committerCarl Hetherington <cth@carlh.net>
Mon, 22 Apr 2024 11:03:04 +0000 (13:03 +0200)
src/lib/hints.cc

index 8e678fdf15987916c48f48593d9a9b1680d28c0a..ca69832a9d3a77d21677d339d7cd473aa08d4ca4 100644 (file)
@@ -166,7 +166,7 @@ void
 Hints::check_unusual_container ()
 {
        auto const film_container = film()->container()->id();
-       if (film_container != "185" && film_container != "239") {
+       if (film()->video_encoding() != VideoEncoding::MPEG2 && film_container != "185" && film_container != "239") {
                hint (_("Your DCP uses an unusual container ratio.  This may cause problems on some projectors.  If possible, use Flat or Scope for the DCP container ratio."));
        }
 }
@@ -756,7 +756,7 @@ void
 Hints::check_8_or_16_audio_channels()
 {
        auto const channels = film()->audio_channels();
-       if (channels != 8 && channels != 16) {
+       if (film()->video_encoding() != VideoEncoding::MPEG2 && channels != 8 && channels != 16) {
                hint(String::compose(_("Your DCP has %1 audio channels, rather than 8 or 16.  This may cause some distributors to raise QC errors when they check your DCP.  To avoid this, set the DCP audio channels to 8 or 16."), channels));
        }
 }