X-Git-Url: https://git.carlh.net/gitweb/?a=blobdiff_plain;f=src%2Flib%2Fhints.h;h=985fa1910d9b10733b88a4f827687bf11de9455a;hb=182b9d2e2feb6545592868606aaf0f0146095481;hp=db7ee49b60007bf12f0ab5336df9efb409f36ce5;hpb=a1546fb6c4e59621d99271b8ca996e96a574f7b3;p=dcpomatic.git

diff --git a/src/lib/hints.h b/src/lib/hints.h
index db7ee49b6..985fa1910 100644
--- a/src/lib/hints.h
+++ b/src/lib/hints.h
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2016-2019 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2016-2021 Carl Hetherington <cth@carlh.net>
 
     This file is part of DCP-o-matic.
 
@@ -18,22 +18,28 @@
 
 */
 
+
+#include "audio_analyser.h"
 #include "signaller.h"
 #include "player_text.h"
 #include "types.h"
 #include "dcp_text_track.h"
 #include "dcpomatic_time.h"
-#include <boost/weak_ptr.hpp>
+#include "weak_film.h"
 #include <boost/signals2.hpp>
+#include <boost/atomic.hpp>
 #include <vector>
 #include <string>
 
+
 class Film;
+class Writer;
+
 
-class Hints : public Signaller, public ExceptionStore
+class Hints : public Signaller, public ExceptionStore, public WeakConstFilm
 {
 public:
-	explicit Hints (boost::weak_ptr<const Film> film);
+	explicit Hints (std::weak_ptr<const Film> film);
 	~Hints ();
 
 	void start ();
@@ -43,18 +49,64 @@ public:
 	boost::signals2::signal<void (void)> Pulse;
 	boost::signals2::signal<void (void)> Finished;
 
+	/* For tests only */
+	void join ();
+	void disable_audio_analysis () {
+		_disable_audio_analysis = true;
+	}
+
 private:
+	friend struct hint_subtitle_too_early;
+
 	void thread ();
 	void hint (std::string h);
-	void text (PlayerText text, TextType type, dcpomatic::DCPTimePeriod period);
+	void audio (std::shared_ptr<AudioBuffers> audio, dcpomatic::DCPTime time);
+	void text (PlayerText text, TextType type, boost::optional<DCPTextTrack> track, dcpomatic::DCPTimePeriod period);
+	void closed_caption (PlayerText text, dcpomatic::DCPTimePeriod period);
+	void open_subtitle (PlayerText text, dcpomatic::DCPTimePeriod period);
+
+	void check_certificates ();
+	void check_interop ();
+	void check_big_font_files ();
+	void check_few_audio_channels ();
+	void check_upmixers ();
+	void check_incorrect_container ();
+	void check_unusual_container ();
+	void check_high_j2k_bandwidth ();
+	void check_frame_rate ();
+	void check_4k_3d ();
+	void check_speed_up ();
+	void check_vob ();
+	void check_3d_in_2d ();
+	bool check_loudness ();
+	void check_ffec_and_ffmc_in_smpte_feature ();
+	void check_out_of_range_markers ();
+	void check_text_languages ();
+	void check_audio_language ();
 
-	boost::weak_ptr<const Film> _film;
 	boost::thread _thread;
+	/** This is used to make a partial DCP containing only the subtitles and closed captions that
+	 *  our final DCP will have.  This means we can see how big the files will be and warn if they
+	 *  will be too big.
+	 */
+	std::shared_ptr<Writer> _writer;
+
+	AudioAnalyser _analyser;
 
-	bool _long_ccap;
-	bool _overlap_ccap;
-	bool _too_many_ccap_lines;
-	boost::optional<dcpomatic::DCPTimePeriod> _last;
+	bool _long_ccap = false;
+	bool _overlap_ccap = false;
+	bool _too_many_ccap_lines = false;
+	boost::optional<dcpomatic::DCPTimePeriod> _last_ccap;
+
+	bool _early_subtitle = false;
+	bool _short_subtitle = false;
+	bool _subtitles_too_close = false;
+	bool _too_many_subtitle_lines = false;
+	bool _long_subtitle = false;
+	bool _very_long_subtitle = false;
+	boost::optional<dcpomatic::DCPTimePeriod> _last_subtitle;
 
 	boost::atomic<bool> _stop;
+
+	bool _disable_audio_analysis = false;
 };