src/lib/hints.cc

   1 /*
   2     Copyright (C) 2016-2022 Carl Hetherington <cth@carlh.net>
   3
   4     This file is part of DCP-o-matic.
   5
   6     DCP-o-matic is free software; you can redistribute it and/or modify
   7     it under the terms of the GNU General Public License as published by
   8     the Free Software Foundation; either version 2 of the License, or
   9     (at your option) any later version.
  10
  11     DCP-o-matic is distributed in the hope that it will be useful,
  12     but WITHOUT ANY WARRANTY; without even the implied warranty of
  13     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  14     GNU General Public License for more details.
  15
  16     You should have received a copy of the GNU General Public License
  17     along with DCP-o-matic.  If not, see <http://www.gnu.org/licenses/>.
  18
  19 */
  20
  21
  22 #include "audio_analysis.h"
  23 #include "audio_content.h"
  24 #include "audio_processor.h"
  25 #include "compose.hpp"
  26 #include "config.h"
  27 #include "content.h"
  28 #include "cross.h"
  29 #include "dcp_content_type.h"
  30 #include "film.h"
  31 #include "font.h"
  32 #include "font_data.h"
  33 #include "hints.h"
  34 #include "player.h"
  35 #include "ratio.h"
  36 #include "text_content.h"
  37 #include "types.h"
  38 #include "util.h"
  39 #include "video_content.h"
  40 #include "writer.h"
  41 #include <dcp/cpl.h>
  42 #include <dcp/raw_convert.h>
  43 #include <dcp/reel.h>
  44 #include <dcp/reel_closed_caption_asset.h>
  45 #include <dcp/reel_subtitle_asset.h>
  46 #include <boost/algorithm/string.hpp>
  47 #include <iostream>
  48
  49 #include "i18n.h"
  50
  51
  52 using std::cout;
  53 using std::make_shared;
  54 using std::max;
  55 using std::shared_ptr;
  56 using std::string;
  57 using std::weak_ptr;
  58 using boost::optional;
  59 using boost::bind;
  60 using namespace dcpomatic;
  61 #if BOOST_VERSION >= 106100
  62 using namespace boost::placeholders;
  63 #endif
  64
  65
  66 /* When checking to see if things are too big, we'll say they are if they
  67  * are more than the target size minus this "slack."
  68  */
  69 #define SIZE_SLACK 4096
  70
  71
  72 /* When writing hints:
  73  * - put quotation marks around the name of a GUI tab that you are referring to (e.g. "DCP" or "DCP→Video" tab)
  74  */
  75
  76
  77 Hints::Hints (weak_ptr<const Film> weak_film)
  78         : WeakConstFilm (weak_film)
  79         , _writer (new Writer(weak_film, weak_ptr<Job>(), true))
  80         , _analyser (film(), film()->playlist(), true, [](float) {})
  81         , _stop (false)
  82 {
  83
  84 }
  85
  86
  87 void
  88 Hints::start ()
  89 {
  90         _thread = boost::thread (bind(&Hints::thread, this));
  91 }
  92
  93
  94 Hints::~Hints ()
  95 {
  96         boost::this_thread::disable_interruption dis;
  97
  98         try {
  99                 _stop = true;
 100                 _thread.interrupt ();
 101                 _thread.join ();
 102         } catch (...) {}
 103 }
 104
 105
 106 void
 107 Hints::check_few_audio_channels ()
 108 {
 109         if (film()->audio_channels() < 6) {
 110                 hint (_("Your DCP has fewer than 6 audio channels.  This may cause problems on some projectors.  You may want to set the DCP to have 6 channels.  It does not matter if your content has fewer channels, as DCP-o-matic will fill the extras with silence."));
 111         }
 112 }
 113
 114
 115 void
 116 Hints::check_upmixers ()
 117 {
 118         auto ap = film()->audio_processor();
 119         if (ap && (ap->id() == "stereo-5.1-upmix-a" || ap->id() == "stereo-5.1-upmix-b")) {
 120                 hint (_("You are using DCP-o-matic's stereo-to-5.1 upmixer.  This is experimental and may result in poor-quality audio.  If you continue, you should listen to the resulting DCP in a cinema to make sure that it sounds good."));
 121         }
 122 }
 123
 124
 125 void
 126 Hints::check_incorrect_container ()
 127 {
 128         int narrower_than_scope = 0;
 129         int scope = 0;
 130         for (auto i: film()->content()) {
 131                 if (i->video) {
 132                         auto const r = Ratio::nearest_from_ratio(i->video->scaled_size(film()->frame_size()).ratio());
 133                         if (r && r->id() == "239") {
 134                                 ++scope;
 135                         } else if (r && r->id() != "239" && r->id() != "235" && r->id() != "190") {
 136                                 ++narrower_than_scope;
 137                         }
 138                 }
 139         }
 140
 141         string const film_container = film()->container()->id();
 142
 143         if (scope && !narrower_than_scope && film_container == "185") {
 144                 hint (_("All of your content is in Scope (2.39:1) but your DCP's container is Flat (1.85:1).  This will letter-box your content inside a Flat (1.85:1) frame.  You may prefer to set your DCP's container to Scope (2.39:1) in the \"DCP\" tab."));
 145         }
 146
 147         if (!scope && narrower_than_scope && film_container == "239") {
 148                 hint (_("All of your content narrower than 1.90:1 but your DCP's container is Scope (2.39:1).  This will pillar-box your content.  You may prefer to set your DCP's container to have the same ratio as your content."));
 149         }
 150 }
 151
 152
 153 void
 154 Hints::check_unusual_container ()
 155 {
 156         auto const film_container = film()->container()->id();
 157         if (film_container != "185" && film_container != "239") {
 158                 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."));
 159         }
 160 }
 161
 162
 163 void
 164 Hints::check_high_j2k_bandwidth ()
 165 {
 166         if (film()->j2k_bandwidth() >= 245000000) {
 167                 hint (_("A few projectors have problems playing back very high bit-rate DCPs.  It is a good idea to drop the JPEG2000 bandwidth down to about 200Mbit/s; this is unlikely to have any visible effect on the image."));
 168         }
 169 }
 170
 171
 172 void
 173 Hints::check_frame_rate ()
 174 {
 175         auto f = film ();
 176         switch (f->video_frame_rate()) {
 177         case 24:
 178                 /* Fine */
 179                 break;
 180         case 25:
 181         {
 182                 /* You might want to go to 24 */
 183                 string base = String::compose(_("You are set up for a DCP at a frame rate of %1 fps.  This frame rate is not supported by all projectors.  You may want to consider changing your frame rate to %2 fps."), 25, 24);
 184                 if (f->interop()) {
 185                         base += "  ";
 186                         base += _("If you do use 25fps you should change your DCP standard to SMPTE.");
 187                 }
 188                 hint (base);
 189                 break;
 190         }
 191         case 30:
 192                 /* 30fps: we can't really offer any decent solutions */
 193                 hint (_("You are set up for a DCP frame rate of 30fps, which is not supported by all projectors.  Be aware that you may have compatibility problems."));
 194                 break;
 195         case 48:
 196         case 50:
 197         case 60:
 198                 /* You almost certainly want to go to half frame rate */
 199                 hint (String::compose(_("You are set up for a DCP at a frame rate of %1 fps.  This frame rate is not supported by all projectors.  You are advised to change the DCP frame rate to %2 fps."), f->video_frame_rate(), f->video_frame_rate() / 2));
 200                 break;
 201         }
 202 }
 203
 204
 205 void
 206 Hints::check_4k_3d ()
 207 {
 208         auto f = film();
 209         if (f->resolution() == Resolution::FOUR_K && f->three_d()) {
 210                 hint (_("4K 3D is only supported by a very limited number of projectors.  Unless you know that you will play this DCP back on a capable projector, it is advisable to set the DCP to be 2K in the \"DCP→Video\" tab."));
 211         }
 212 }
 213
 214
 215 void
 216 Hints::check_speed_up ()
 217 {
 218         optional<double> lowest_speed_up;
 219         optional<double> highest_speed_up;
 220         for (auto i: film()->content()) {
 221                 double spu = film()->active_frame_rate_change(i->position()).speed_up;
 222                 if (!lowest_speed_up || spu < *lowest_speed_up) {
 223                         lowest_speed_up = spu;
 224                 }
 225                 if (!highest_speed_up || spu > *highest_speed_up) {
 226                         highest_speed_up = spu;
 227                 }
 228         }
 229
 230         double worst_speed_up = 1;
 231         if (highest_speed_up) {
 232                 worst_speed_up = *highest_speed_up;
 233         }
 234         if (lowest_speed_up) {
 235                 worst_speed_up = max (worst_speed_up, 1 / *lowest_speed_up);
 236         }
 237
 238         if (worst_speed_up > 25.5/24.0) {
 239                 hint (_("There is a large difference between the frame rate of your DCP and that of some of your content.  This will cause your audio to play back at a much lower or higher pitch than it should.  You are advised to set your DCP frame rate to one closer to your content, provided that your target projection systems support your chosen DCP rate."));
 240         }
 241
 242 }
 243
 244
 245 void
 246 Hints::check_interop ()
 247 {
 248         if (film()->interop()) {
 249                 hint (_("In general it is now advisable to make SMPTE DCPs unless you have a particular reason to use Interop.  You are advised to set your DCP to use the SMPTE standard in the \"DCP\" tab."));
 250         }
 251 }
 252
 253
 254 void
 255 Hints::check_big_font_files ()
 256 {
 257         bool big_font_files = false;
 258         if (film()->interop ()) {
 259                 for (auto i: film()->content()) {
 260                         for (auto j: i->text) {
 261                                 for (auto k: j->fonts()) {
 262                                         auto const p = k->file ();
 263                                         if (p && boost::filesystem::file_size(p.get()) >= (MAX_FONT_FILE_SIZE - SIZE_SLACK)) {
 264                                                 big_font_files = true;
 265                                         }
 266                                 }
 267                         }
 268                 }
 269         }
 270
 271         if (big_font_files) {
 272                 hint (_("You have specified a font file which is larger than 640kB.  This is very likely to cause problems on playback."));
 273         }
 274 }
 275
 276
 277 void
 278 Hints::check_vob ()
 279 {
 280         int vob = 0;
 281         for (auto i: film()->content()) {
 282                 if (boost::algorithm::starts_with(i->path(0).filename().string(), "VTS_")) {
 283                         ++vob;
 284                 }
 285         }
 286
 287         if (vob > 1) {
 288                 hint (String::compose (_("You have %1 files that look like they are VOB files from DVD. You should join them to ensure smooth joins between the files."), vob));
 289         }
 290 }
 291
 292
 293 void
 294 Hints::check_3d_in_2d ()
 295 {
 296         int three_d = 0;
 297         for (auto i: film()->content()) {
 298                 if (i->video && i->video->frame_type() != VideoFrameType::TWO_D) {
 299                         ++three_d;
 300                 }
 301         }
 302
 303         if (three_d > 0 && !film()->three_d()) {
 304                 hint (_("You are using 3D content but your DCP is set to 2D.  Set the DCP to 3D if you want to play it back on a 3D system (e.g. Real-D, MasterImage etc.)"));
 305         }
 306 }
 307
 308
 309 /** @return true if the loudness could be checked, false if it could not because no analysis was available */
 310 bool
 311 Hints::check_loudness ()
 312 {
 313         auto path = film()->audio_analysis_path(film()->playlist());
 314         if (!boost::filesystem::exists(path)) {
 315                 return false;
 316         }
 317
 318         try {
 319                 auto an = make_shared<AudioAnalysis>(path);
 320
 321                 string ch;
 322
 323                 auto sample_peak = an->sample_peak ();
 324                 auto true_peak = an->true_peak ();
 325
 326                 for (size_t i = 0; i < sample_peak.size(); ++i) {
 327                         float const peak = max (sample_peak[i].peak, true_peak.empty() ? 0 : true_peak[i]);
 328                         float const peak_dB = linear_to_db(peak) + an->gain_correction(film()->playlist());
 329                         if (peak_dB > -3) {
 330                                 ch += dcp::raw_convert<string>(short_audio_channel_name(i)) + ", ";
 331                         }
 332                 }
 333
 334                 ch = ch.substr (0, ch.length() - 2);
 335
 336                 if (!ch.empty()) {
 337                         hint(String::compose(
 338                                         _("Your audio level is very high (on %1).  You should reduce the gain of your audio content."),
 339                                         ch
 340                                         )
 341                              );
 342                 }
 343         } catch (OldFormatError& e) {
 344                 /* The audio analysis is too old to load in */
 345                 return false;
 346         }
 347
 348         return true;
 349 }
 350
 351
 352 static
 353 bool
 354 subtitle_mxf_too_big (shared_ptr<dcp::SubtitleAsset> asset)
 355 {
 356         return asset && asset->file() && boost::filesystem::file_size(*asset->file()) >= (MAX_TEXT_MXF_SIZE - SIZE_SLACK);
 357 }
 358
 359
 360 void
 361 Hints::check_out_of_range_markers ()
 362 {
 363         auto const length = film()->length();
 364         for (auto const& i: film()->markers()) {
 365                 if (i.second >= length) {
 366                         hint (_("At least one marker comes after the end of the project and will be ignored."));
 367                 }
 368         }
 369 }
 370
 371
 372 void
 373 Hints::thread ()
 374 try
 375 {
 376         start_of_thread ("Hints");
 377
 378         auto film = _film.lock ();
 379         if (!film) {
 380                 return;
 381         }
 382
 383         auto content = film->content ();
 384
 385         check_certificates ();
 386         check_interop ();
 387         check_big_font_files ();
 388         check_few_audio_channels ();
 389         check_upmixers ();
 390         check_incorrect_container ();
 391         check_unusual_container ();
 392         check_high_j2k_bandwidth ();
 393         check_frame_rate ();
 394         check_4k_3d ();
 395         check_speed_up ();
 396         check_vob ();
 397         check_3d_in_2d ();
 398         auto const check_loudness_done = check_loudness ();
 399         check_ffec_and_ffmc_in_smpte_feature ();
 400         check_out_of_range_markers ();
 401         check_text_languages ();
 402         check_audio_language ();
 403
 404         if (check_loudness_done) {
 405                 emit (bind(boost::ref(Progress), _("Examining subtitles and closed captions")));
 406         } else {
 407                 emit (bind(boost::ref(Progress), _("Examining audio, subtitles and closed captions")));
 408         }
 409
 410         auto player = make_shared<Player>(film, Image::Alignment::COMPACT);
 411         player->set_ignore_video ();
 412         if (check_loudness_done || _disable_audio_analysis) {
 413                 /* We don't need to analyse audio because we already loaded a suitable analysis */
 414                 player->set_ignore_audio ();
 415         }
 416         player->Audio.connect (bind(&Hints::audio, this, _1, _2));
 417         player->Text.connect (bind(&Hints::text, this, _1, _2, _3, _4));
 418
 419         struct timeval last_pulse;
 420         gettimeofday (&last_pulse, 0);
 421
 422         while (!player->pass()) {
 423
 424                 struct timeval now;
 425                 gettimeofday (&now, 0);
 426                 if ((seconds(now) - seconds(last_pulse)) > 1) {
 427                         if (_stop) {
 428                                 return;
 429                         }
 430                         emit (bind (boost::ref(Pulse)));
 431                         last_pulse = now;
 432                 }
 433         }
 434
 435         if (!check_loudness_done) {
 436                 _analyser.finish ();
 437                 _analyser.get().write(film->audio_analysis_path(film->playlist()));
 438                 check_loudness ();
 439         }
 440
 441         _writer->write (player->get_subtitle_fonts());
 442
 443         if (_long_subtitle && !_very_long_subtitle) {
 444                 hint (_("At least one of your subtitle lines has more than 52 characters.  It is recommended to make each line 52 characters at most in length."));
 445         } else if (_very_long_subtitle) {
 446                 hint (_("At least one of your subtitle lines has more than 79 characters.  You should make each line 79 characters at most in length."));
 447         }
 448
 449         bool ccap_xml_too_big = false;
 450         bool ccap_mxf_too_big = false;
 451         bool subs_mxf_too_big = false;
 452
 453         auto dcp_dir = film->dir("hints") / dcpomatic::get_process_id();
 454         boost::filesystem::remove_all (dcp_dir);
 455
 456         _writer->finish (film->dir("hints") / dcpomatic::get_process_id());
 457
 458         dcp::DCP dcp (dcp_dir);
 459         dcp.read ();
 460         DCPOMATIC_ASSERT (dcp.cpls().size() == 1);
 461         for (auto reel: dcp.cpls()[0]->reels()) {
 462                 for (auto ccap: reel->closed_captions()) {
 463                         if (ccap->asset() && ccap->asset()->xml_as_string().length() > static_cast<size_t>(MAX_CLOSED_CAPTION_XML_SIZE - SIZE_SLACK) && !ccap_xml_too_big) {
 464                                 hint (_(
 465                                                 "At least one of your closed caption files' XML part is larger than " MAX_CLOSED_CAPTION_XML_SIZE_TEXT
 466                                                 ".  You should divide the DCP into shorter reels."
 467                                        ));
 468                                 ccap_xml_too_big = true;
 469                         }
 470                         if (subtitle_mxf_too_big(ccap->asset()) && !ccap_mxf_too_big) {
 471                                 hint (_(
 472                                                 "At least one of your closed caption files is larger than " MAX_TEXT_MXF_SIZE_TEXT
 473                                                 " in total.  You should divide the DCP into shorter reels."
 474                                        ));
 475                                 ccap_mxf_too_big = true;
 476                         }
 477                 }
 478                 if (reel->main_subtitle() && subtitle_mxf_too_big(reel->main_subtitle()->asset()) && !subs_mxf_too_big) {
 479                         hint (_(
 480                                         "At least one of your subtitle files is larger than " MAX_TEXT_MXF_SIZE_TEXT " in total.  "
 481                                         "You should divide the DCP into shorter reels."
 482                                ));
 483                         subs_mxf_too_big = true;
 484                 }
 485         }
 486         boost::filesystem::remove_all (dcp_dir);
 487
 488         emit (bind(boost::ref(Finished)));
 489 }
 490 catch (boost::thread_interrupted)
 491 {
 492         /* The Hints object is being destroyed before it has finished, so just give up */
 493 }
 494 catch (...)
 495 {
 496         store_current ();
 497 }
 498
 499
 500 void
 501 Hints::hint (string h)
 502 {
 503         emit(bind(boost::ref(Hint), h));
 504 }
 505
 506
 507 void
 508 Hints::audio (shared_ptr<AudioBuffers> audio, DCPTime time)
 509 {
 510         _analyser.analyse (audio, time);
 511 }
 512
 513
 514 void
 515 Hints::text (PlayerText text, TextType type, optional<DCPTextTrack> track, DCPTimePeriod period)
 516 {
 517         _writer->write (text, type, track, period);
 518
 519         switch (type) {
 520         case TextType::CLOSED_CAPTION:
 521                 closed_caption (text, period);
 522                 break;
 523         case TextType::OPEN_SUBTITLE:
 524                 open_subtitle (text, period);
 525                 break;
 526         default:
 527                 break;
 528         }
 529 }
 530
 531
 532 void
 533 Hints::closed_caption (PlayerText text, DCPTimePeriod period)
 534 {
 535         int lines = text.string.size();
 536         for (auto i: text.string) {
 537                 if (utf8_strlen(i.text()) > MAX_CLOSED_CAPTION_LENGTH) {
 538                         ++lines;
 539                         if (!_long_ccap) {
 540                                 _long_ccap = true;
 541                                 hint (
 542                                         String::compose(
 543                                                 "At least one of your closed caption lines has more than %1 characters.  "
 544                                                 "It is advisable to make each line %1 characters at most in length.",
 545                                                 MAX_CLOSED_CAPTION_LENGTH,
 546                                                 MAX_CLOSED_CAPTION_LENGTH)
 547                                      );
 548                         }
 549                 }
 550         }
 551
 552         if (!_too_many_ccap_lines && lines > MAX_CLOSED_CAPTION_LINES) {
 553                 hint (String::compose(_("Some of your closed captions span more than %1 lines, so they will be truncated."), MAX_CLOSED_CAPTION_LINES));
 554                 _too_many_ccap_lines = true;
 555         }
 556
 557         /* XXX: maybe overlapping closed captions (i.e. different languages) are OK with Interop? */
 558         if (film()->interop() && !_overlap_ccap && _last_ccap && _last_ccap->overlap(period)) {
 559                 _overlap_ccap = true;
 560                 hint (_("You have overlapping closed captions, which are not allowed in Interop DCPs.  Change your DCP standard to SMPTE."));
 561         }
 562
 563         _last_ccap = period;
 564 }
 565
 566
 567 void
 568 Hints::open_subtitle (PlayerText text, DCPTimePeriod period)
 569 {
 570         if (period.from < DCPTime::from_seconds(4) && !_early_subtitle) {
 571                 _early_subtitle = true;
 572                 hint (_("It is advisable to put your first subtitle at least 4 seconds after the start of the DCP to make sure it is seen."));
 573         }
 574
 575         int const vfr = film()->video_frame_rate ();
 576
 577         if (period.duration().frames_round(vfr) < 15 && !_short_subtitle) {
 578                 _short_subtitle = true;
 579                 hint (_("At least one of your subtitles lasts less than 15 frames.  It is advisable to make each subtitle at least 15 frames long."));
 580         }
 581
 582         if (_last_subtitle && DCPTime(period.from - _last_subtitle->to).frames_round(vfr) < 2 && !_subtitles_too_close) {
 583                 _subtitles_too_close = true;
 584                 hint (_("At least one of your subtitles starts less than 2 frames after the previous one.  It is advisable to make the gap between subtitles at least 2 frames."));
 585         }
 586
 587         if (text.string.size() > 3 && !_too_many_subtitle_lines) {
 588                 _too_many_subtitle_lines = true;
 589                 hint (_("At least one of your subtitles has more than 3 lines.  It is advisable to use no more than 3 lines."));
 590         }
 591
 592         size_t longest_line = 0;
 593         for (auto const& i: text.string) {
 594                 longest_line = max (longest_line, i.text().length());
 595         }
 596
 597         if (longest_line > 52) {
 598                 _long_subtitle = true;
 599         }
 600
 601         if (longest_line > 79) {
 602                 _very_long_subtitle = true;
 603         }
 604
 605         _last_subtitle = period;
 606 }
 607
 608
 609 void
 610 Hints::check_ffec_and_ffmc_in_smpte_feature ()
 611 {
 612         auto f = film();
 613         if (!f->interop() && f->dcp_content_type()->libdcp_kind() == dcp::ContentKind::FEATURE && (!f->marker(dcp::Marker::FFEC) || !f->marker(dcp::Marker::FFMC))) {
 614                 hint (_("SMPTE DCPs with the type FTR (feature) should have markers for the first frame of end credits (FFEC) and the first frame of moving credits (FFMC).  You should add these markers using the 'Markers' button in the \"DCP\" tab."));
 615         }
 616 }
 617
 618
 619 void
 620 Hints::join ()
 621 {
 622         _thread.join ();
 623 }
 624
 625
 626 void
 627 Hints::check_text_languages ()
 628 {
 629         for (auto i: film()->content()) {
 630                 for (auto j: i->text) {
 631                         if (j->use() && !j->language()) {
 632                                 hint (_("At least one piece of subtitle or closed caption content has no specified language.  "
 633                                         "It is advisable to set the language for each piece of subtitle or closed caption content "
 634                                         "in the \"Content→Timed text\", \"Content→Open subtitles\" or \"Content→Closed captions\" tab."));
 635                                 return;
 636                         }
 637                 }
 638         }
 639 }
 640
 641
 642 void
 643 Hints::check_audio_language ()
 644 {
 645         auto content = film()->content();
 646         auto mapped_audio =
 647                 std::find_if(content.begin(), content.end(), [](shared_ptr<const Content> c) {
 648                         return c->audio && !c->audio->mapping().mapped_output_channels().empty();
 649                 });
 650
 651         if (mapped_audio != content.end() && !film()->audio_language()) {
 652                 hint (_("Some of your content has audio but you have not set the audio language.  It is advisable to set the audio language "
 653                         "in the \"DCP\" tab unless your audio has no spoken parts."));
 654         }
 655 }
 656
 657
 658 void
 659 Hints::check_certificates ()
 660 {
 661         auto bad = Config::instance()->check_certificates();
 662         if (!bad) {
 663                 return;
 664         }
 665
 666         switch (*bad) {
 667         case Config::BAD_SIGNER_UTF8_STRINGS:
 668                 hint(_("The certificate chain that DCP-o-matic uses for signing DCPs and KDMs contains a small error "
 669                        "which will prevent DCPs from being validated correctly on some systems.  You are advised to "
 670                        "re-create the signing certificate chain by clicking the \"Re-make certificates and key...\" "
 671                        "button in the Keys page of Preferences."));
 672                 break;
 673         case Config::BAD_SIGNER_VALIDITY_TOO_LONG:
 674                 hint(_("The certificate chain that DCP-o-matic uses for signing DCPs and KDMs has a validity period "
 675                        "that is too long.  This will cause problems playing back DCPs on some systems. "
 676                        "You are advised to re-create the signing certificate chain by clicking the "
 677                        "\"Re-make certificates and key...\" button in the Keys page of Preferences."));
 678                 break;
 679         default:
 680                 /* Some bad situations can't happen here as DCP-o-matic would have refused to start until they are fixed */
 681                 break;
 682         }
 683 }
 684