Add some missing override labels.
authorCarl Hetherington <cth@carlh.net>
Sat, 9 Apr 2022 11:58:27 +0000 (13:58 +0200)
committerCarl Hetherington <cth@carlh.net>
Sat, 9 Apr 2022 11:58:27 +0000 (13:58 +0200)
64 files changed:
src/lib/analytics.h
src/lib/audio_decoder.h
src/lib/check_content_change_job.h
src/lib/combine_dcp_job.h
src/lib/config.h
src/lib/curl_uploader.h
src/lib/dcp_content.h
src/lib/dcp_subtitle_content.h
src/lib/dcp_subtitle_decoder.h
src/lib/dkdm_recipient.h
src/lib/dkdm_wrapper.h
src/lib/encode_server.h
src/lib/encoded_log_entry.h
src/lib/examine_content_job.h
src/lib/examine_ffmpeg_subtitles_job.h
src/lib/ffmpeg_content.h
src/lib/ffmpeg_image_proxy.h
src/lib/file_log.h
src/lib/image_content.h
src/lib/j2k_image_proxy.h
src/lib/mid_side_decoder.h
src/lib/null_log.h
src/lib/raw_image_proxy.h
src/lib/screen.h
src/lib/send_kdm_email_job.h
src/lib/stdout_log.h
src/lib/string_log_entry.h
src/lib/string_text_file_content.h
src/lib/string_text_file_decoder.h
src/lib/text_decoder.h
src/lib/upload_job.h
src/lib/upmixer_a.h
src/lib/upmixer_b.h
src/lib/video_decoder.h
src/tools/dcpomatic.cc
src/tools/dcpomatic_batch.cc
src/tools/dcpomatic_disk.cc
src/tools/dcpomatic_kdm.cc
src/tools/dcpomatic_player.cc
src/tools/dcpomatic_playlist.cc
src/tools/dcpomatic_server.cc
src/wx/audio_dialog.h
src/wx/barco_alchemy_certificate_panel.h
src/wx/batch_job_view.h
src/wx/check_box.h
src/wx/christie_certificate_panel.h
src/wx/config_dialog.h
src/wx/content_panel.h
src/wx/content_view.h
src/wx/credentials_download_certificate_panel.h
src/wx/dcpomatic_button.h
src/wx/full_config_dialog.cc
src/wx/full_language_tag_dialog.cc
src/wx/gdc_certificate_panel.h
src/wx/normal_job_view.h
src/wx/player_config_dialog.cc
src/wx/playlist_controls.h
src/wx/standard_controls.h
src/wx/static_text.h
src/wx/time_picker.h
src/wx/timeline_text_content_view.h
src/wx/timeline_time_axis_view.h
src/wx/wx_signal_manager.h
test/job_manager_test.cc

index c0a3e17eff1776c1fb633c085a4c4038ceb26279..0de0b51caaa75b13d9b4ce69036e4aebdb571fd7 100644 (file)
@@ -31,8 +31,8 @@ public:
 
        void successful_dcp_encode ();
 
-       void write () const;
-       void read ();
+       void write () const override;
+       void read () override;
 
        boost::signals2::signal<void (std::string, std::string)> Message;
 
index a2e06f11ceed15773f9987eb96e3017eddca1ed0..a8495aaa8b16cdd1090f09923f6f4084bc43ffa0 100644 (file)
@@ -51,9 +51,9 @@ class AudioDecoder : public std::enable_shared_from_this<AudioDecoder>, public D
 public:
        AudioDecoder (Decoder* parent, std::shared_ptr<const AudioContent> content, bool fast);
 
-       boost::optional<dcpomatic::ContentTime> position (std::shared_ptr<const Film> film) const;
+       boost::optional<dcpomatic::ContentTime> position (std::shared_ptr<const Film> film) const override;
        void emit (std::shared_ptr<const Film> film, AudioStreamPtr stream, std::shared_ptr<const AudioBuffers>, dcpomatic::ContentTime, bool time_already_delayed = false);
-       void seek ();
+       void seek () override;
        void flush ();
 
        dcpomatic::ContentTime stream_position (std::shared_ptr<const Film> film, AudioStreamPtr stream) const;
index b3cdf594c1076d762bef89ea43a3e8fcf228e7f7..a374e2a6fb373783e286afd41ff7b29e3092872d 100644 (file)
@@ -30,7 +30,7 @@ public:
        CheckContentChangeJob (std::shared_ptr<const Film>);
        ~CheckContentChangeJob ();
 
-       std::string name () const;
-       std::string json_name () const;
-       void run ();
+       std::string name () const override;
+       std::string json_name () const override;
+       void run () override;
 };
index 4a7e02b8b4e2d055c3947748d52c8ddc5a12b8a1..5311ddd3e956cc815d1215ff02d8f6772fedd4a0 100644 (file)
@@ -28,9 +28,9 @@ class CombineDCPJob : public Job
 public:
        CombineDCPJob (std::vector<boost::filesystem::path> inputs, boost::filesystem::path output, std::string annotation_text);
 
-       std::string name () const;
-       std::string json_name () const;
-       void run ();
+       std::string name () const override;
+       std::string json_name () const override;
+       void run () override;
 
 private:
        std::vector<boost::filesystem::path> _inputs;
index 29f401755b06cf814db5c1cb2207848099053d9e..e9b3ec203af95e762f6271ef66aa501f36ab5a1d 100644 (file)
@@ -1108,7 +1108,7 @@ public:
 
        static boost::signals2::signal<bool (BadReason)> Bad;
 
-       void write () const;
+       void write () const override;
        void write_config () const;
        void write_cinemas () const;
        void write_dkdm_recipients () const;
@@ -1135,7 +1135,7 @@ public:
 
 private:
        Config ();
-       void read ();
+       void read () override;
        void set_defaults ();
        void set_kdm_email_to_default ();
        void set_notification_email_to_default ();
index a3ef3cb229ba6367e622fb4a0402b6bbd35d2a24..564e774934c1f16afb555886d9926e5e15b1661a 100644 (file)
@@ -32,8 +32,8 @@ public:
        size_t read_callback (void* ptr, size_t size, size_t nmemb);
 
 protected:
-       virtual void create_directory (boost::filesystem::path directory);
-       virtual void upload_file (boost::filesystem::path from, boost::filesystem::path to, boost::uintmax_t& transferred, boost::uintmax_t total_size);
+       void create_directory (boost::filesystem::path directory) override;
+       void upload_file (boost::filesystem::path from, boost::filesystem::path to, boost::uintmax_t& transferred, boost::uintmax_t total_size) override;
 
 private:
        CURL* _curl;
index 40ed181fe7f383d6489f2e14c9d88e2d2b19b788..d57c0059c753cdf621d1e9d11c7dbac02b25d82b 100644 (file)
@@ -65,18 +65,18 @@ public:
                return std::dynamic_pointer_cast<const DCPContent> (Content::shared_from_this ());
        }
 
-       dcpomatic::DCPTime full_length (std::shared_ptr<const Film> film) const;
-       dcpomatic::DCPTime approximate_length () const;
+       dcpomatic::DCPTime full_length (std::shared_ptr<const Film> film) const override;
+       dcpomatic::DCPTime approximate_length () const override;
 
-       void examine (std::shared_ptr<const Film> film, std::shared_ptr<Job>);
-       std::string summary () const;
-       std::string technical_summary () const;
-       void as_xml (xmlpp::Node *, bool with_paths) const;
-       std::string identifier () const;
-       void take_settings_from (std::shared_ptr<const Content> c);
+       void examine (std::shared_ptr<const Film> film, std::shared_ptr<Job>) override;
+       std::string summary () const override;
+       std::string technical_summary () const override;
+       void as_xml (xmlpp::Node *, bool with_paths) const override;
+       std::string identifier () const override;
+       void take_settings_from (std::shared_ptr<const Content> c) override;
 
        void set_default_colour_conversion ();
-       std::list<dcpomatic::DCPTime> reel_split_points (std::shared_ptr<const Film> film) const;
+       std::list<dcpomatic::DCPTime> reel_split_points (std::shared_ptr<const Film> film) const override;
 
        std::vector<boost::filesystem::path> directories () const;
 
@@ -173,7 +173,7 @@ public:
 private:
        friend struct reels_test5;
 
-       void add_properties (std::shared_ptr<const Film> film, std::list<UserProperty>& p) const;
+       void add_properties (std::shared_ptr<const Film> film, std::list<UserProperty>& p) const override;
 
        void read_directory (boost::filesystem::path);
        void read_sub_directory (boost::filesystem::path);
index f4a767d210c6c587630c99132db208274c7fd122..5949f8b0ba5ef866b68a418379ff98c182b0053f 100644 (file)
@@ -27,12 +27,12 @@ public:
        DCPSubtitleContent (boost::filesystem::path);
        DCPSubtitleContent (cxml::ConstNodePtr, int);
 
-       void examine (std::shared_ptr<const Film> film, std::shared_ptr<Job>);
-       std::string summary () const;
-       std::string technical_summary () const;
-       void as_xml (xmlpp::Node *, bool with_paths) const;
-       dcpomatic::DCPTime full_length (std::shared_ptr<const Film> film) const;
-       dcpomatic::DCPTime approximate_length () const;
+       void examine (std::shared_ptr<const Film> film, std::shared_ptr<Job>) override;
+       std::string summary () const override;
+       std::string technical_summary () const override;
+       void as_xml (xmlpp::Node *, bool with_paths) const override;
+       dcpomatic::DCPTime full_length (std::shared_ptr<const Film> film) const override;
+       dcpomatic::DCPTime approximate_length () const override;
 
 private:
        dcpomatic::ContentTime _length;
index 95e783d0612a9876ec5f40050240bbd1e3bb03af..4a54aaf2ef09529ead0f04f29b57eda8423a6778 100644 (file)
@@ -32,10 +32,10 @@ class DCPSubtitleDecoder : public DCPSubtitle, public Decoder
 public:
        DCPSubtitleDecoder (std::shared_ptr<const Film> film, std::shared_ptr<const DCPSubtitleContent>);
 
-       bool pass ();
-       void seek (dcpomatic::ContentTime time, bool accurate);
+       bool pass () override;
+       void seek (dcpomatic::ContentTime time, bool accurate) override;
 
-       std::vector<dcpomatic::FontData> fonts () const;
+       std::vector<dcpomatic::FontData> fonts () const override;
 
 private:
        dcpomatic::ContentTimePeriod content_time_period (std::shared_ptr<const dcp::Subtitle> s) const;
index 50f2f3e15277ad6fba80d742c823aff35ff23508..6e9e4dfb1721bdbb0fdc19dd2954e1f57b53eb25 100644 (file)
@@ -47,7 +47,7 @@ public:
 
        explicit DKDMRecipient (cxml::ConstNodePtr);
 
-       void as_xml (xmlpp::Element *) const;
+       void as_xml (xmlpp::Element *) const override;
 
        std::list<std::string> emails;
        int utc_offset_hour;
index 0c2992bc41f6f1d413b8eefce6a90b3a1a3a80f6..7227fdc86449ea4ccf9b28fdb8c803fe60878b1c 100644 (file)
@@ -61,8 +61,8 @@ public:
                : _dkdm (k)
        {}
 
-       std::string name () const;
-       void as_xml (xmlpp::Element *) const;
+       std::string name () const override;
+       void as_xml (xmlpp::Element *) const override;
 
        dcp::EncryptedKDM dkdm () const {
                return _dkdm;
@@ -80,11 +80,11 @@ public:
                : _name (name)
        {}
 
-       std::string name () const {
+       std::string name () const override {
                return _name;
        }
 
-       void as_xml (xmlpp::Element *) const;
+       void as_xml (xmlpp::Element *) const override;
 
        std::list<std::shared_ptr<DKDMBase>> children () const {
                return _children;
index 0ca60f3f3debf8c0b423618dcccc9d0f908d3955..f93d667467a0a4fe439651fe540d7dd3107e68fe 100644 (file)
@@ -51,10 +51,10 @@ public:
        EncodeServer (bool verbose, int num_threads);
        ~EncodeServer ();
 
-       void run ();
+       void run () override;
 
 private:
-       void handle (std::shared_ptr<Socket>);
+       void handle (std::shared_ptr<Socket>) override;
        void worker_thread ();
        int process (std::shared_ptr<Socket> socket, struct timeval &, struct timeval &);
        void broadcast_thread ();
index c4633dbe9dd2b571f4c37d3750c079fb4f2e8763..fadf48add6bfd37334cc2d886f5758f3b04b573a 100644 (file)
@@ -27,7 +27,7 @@ class EncodedLogEntry : public LogEntry
 public:
        EncodedLogEntry (int frame, std::string ip, double receive, double encode, double send);
 
-       std::string message () const;
+       std::string message () const override;
 
 private:
        int _frame;
index 9437942dc0aa659e748c8d507fe2d75e776b40ef..601b285844a0466dd66f5e652cb582602028ea8f 100644 (file)
@@ -31,9 +31,9 @@ public:
        ExamineContentJob (std::shared_ptr<const Film>, std::shared_ptr<Content>);
        ~ExamineContentJob ();
 
-       std::string name () const;
-       std::string json_name () const;
-       void run ();
+       std::string name () const override;
+       std::string json_name () const override;
+       void run () override;
 
        std::shared_ptr<Content> content () const {
                return _content;
index c9c0cabf4dbd9c316a90fb1713d4ab12fafaf385..1a065df57b485c95264ddba4a8a065d49b86791d 100644 (file)
@@ -29,9 +29,9 @@ public:
        ExamineFFmpegSubtitlesJob (std::shared_ptr<const Film>, std::shared_ptr<FFmpegContent>);
        ~ExamineFFmpegSubtitlesJob ();
 
-       std::string name () const;
-       std::string json_name () const;
-       void run ();
+       std::string name () const override;
+       std::string json_name () const override;
+       void run () override;
 
 private:
        std::shared_ptr<FFmpegContent> _content;
index df7b56edca41e9ecd5475c2f0a9658f1a1b855e5..ce4a8aa6985cdbdc38af33bf5e4c8b479cbc65db 100644 (file)
@@ -59,15 +59,15 @@ public:
                return std::dynamic_pointer_cast<const FFmpegContent> (Content::shared_from_this ());
        }
 
-       void examine (std::shared_ptr<const Film> film, std::shared_ptr<Job>);
-       void take_settings_from (std::shared_ptr<const Content> c);
-       std::string summary () const;
-       std::string technical_summary () const;
-       void as_xml (xmlpp::Node *, bool with_paths) const;
-       dcpomatic::DCPTime full_length (std::shared_ptr<const Film> film) const;
-       dcpomatic::DCPTime approximate_length () const;
+       void examine (std::shared_ptr<const Film> film, std::shared_ptr<Job>) override;
+       void take_settings_from (std::shared_ptr<const Content> c) override;
+       std::string summary () const override;
+       std::string technical_summary () const override;
+       void as_xml (xmlpp::Node *, bool with_paths) const override;
+       dcpomatic::DCPTime full_length (std::shared_ptr<const Film> film) const override;
+       dcpomatic::DCPTime approximate_length () const override;
 
-       std::string identifier () const;
+       std::string identifier () const override;
 
        void set_default_colour_conversion ();
 
@@ -100,7 +100,7 @@ public:
        void signal_subtitle_stream_changed ();
 
 private:
-       void add_properties (std::shared_ptr<const Film> film, std::list<UserProperty> &) const;
+       void add_properties (std::shared_ptr<const Film> film, std::list<UserProperty> &) const override;
 
        friend struct ffmpeg_pts_offset_test;
        friend struct audio_sampling_rate_test;
index 48317ed75c7ae16d3d3d61c30e729a9ec9c4976c..b2b0260b5437eab63ae99bc66e8d30ad76349ddd 100644 (file)
@@ -34,12 +34,12 @@ public:
        Result image (
                Image::Alignment alignment,
                boost::optional<dcp::Size> size = boost::optional<dcp::Size> ()
-               ) const;
+               ) const override;
 
-       void add_metadata (xmlpp::Node *) const;
-       void write_to_socket (std::shared_ptr<Socket>) const;
-       bool same (std::shared_ptr<const ImageProxy> other) const;
-       size_t memory_used () const;
+       void add_metadata (xmlpp::Node *) const override;
+       void write_to_socket (std::shared_ptr<Socket>) const override;
+       bool same (std::shared_ptr<const ImageProxy> other) const override;
+       size_t memory_used () const override;
 
        int avio_read (uint8_t* buffer, int const amount);
        int64_t avio_seek (int64_t const pos, int whence);
index d7667f799ba37cd40e1aecb5c8aa81dd5c09db9f..2795349ce6522dbb39c4804157368b201bfa1c58 100644 (file)
@@ -28,10 +28,10 @@ public:
        explicit FileLog (boost::filesystem::path file);
        FileLog (boost::filesystem::path file, int types);
 
-       std::string head_and_tail (int amount = 1024) const;
+       std::string head_and_tail (int amount = 1024) const override;
 
 private:
-       void do_log (std::shared_ptr<const LogEntry> entry);
+       void do_log (std::shared_ptr<const LogEntry> entry) override;
 
        /** filename to write to */
        boost::filesystem::path _file;
index 81396c63ba19a95683c856d2441d17f62db80306..d817eeee84e4cc0b6a2685666db207eae79a0025 100644 (file)
@@ -37,21 +37,21 @@ public:
                return std::dynamic_pointer_cast<const ImageContent> (Content::shared_from_this ());
        };
 
-       void examine (std::shared_ptr<const Film> film, std::shared_ptr<Job>);
-       std::string summary () const;
-       std::string technical_summary () const;
-       void as_xml (xmlpp::Node *, bool with_paths) const;
-       dcpomatic::DCPTime full_length (std::shared_ptr<const Film> film) const;
-       dcpomatic::DCPTime approximate_length () const;
+       void examine (std::shared_ptr<const Film> film, std::shared_ptr<Job>) override;
+       std::string summary () const override;
+       std::string technical_summary () const override;
+       void as_xml (xmlpp::Node *, bool with_paths) const override;
+       dcpomatic::DCPTime full_length (std::shared_ptr<const Film> film) const override;
+       dcpomatic::DCPTime approximate_length () const override;
 
-       std::string identifier () const;
+       std::string identifier () const override;
 
        void set_default_colour_conversion ();
 
        bool still () const;
 
 private:
-       void add_properties (std::shared_ptr<const Film> film, std::list<UserProperty>& p) const;
+       void add_properties (std::shared_ptr<const Film> film, std::list<UserProperty>& p) const override;
 
        boost::optional<boost::filesystem::path> _path_to_scan;
 };
index d925bef864fe9129fc59dd8246b4b305325ea55e..9666ea406784f9981780661e60ba39c7dcfb2c31 100644 (file)
@@ -59,13 +59,13 @@ public:
        Result image (
                Image::Alignment alignment,
                boost::optional<dcp::Size> size = boost::optional<dcp::Size> ()
-               ) const;
+               ) const override;
 
-       void add_metadata (xmlpp::Node *) const;
-       void write_to_socket (std::shared_ptr<Socket>) const;
+       void add_metadata (xmlpp::Node *) const override;
+       void write_to_socket (std::shared_ptr<Socket> override) const override;
        /** @return true if our image is definitely the same as another, false if it is probably not */
-       bool same (std::shared_ptr<const ImageProxy>) const;
-       int prepare (Image::Alignment alignment, boost::optional<dcp::Size> = boost::optional<dcp::Size>()) const;
+       bool same (std::shared_ptr<const ImageProxy>) const override;
+       int prepare (Image::Alignment alignment, boost::optional<dcp::Size> = boost::optional<dcp::Size>()) const override;
 
        std::shared_ptr<const dcp::Data> j2k () const {
                return _data;
@@ -79,7 +79,7 @@ public:
                return _eye;
        }
 
-       size_t memory_used () const;
+       size_t memory_used () const override;
 
 private:
        std::shared_ptr<const dcp::Data> _data;
index 99821caa463985c12c4067e13cc293f204eabfd6..857975478074f31c936d4d071b9e031d7c07b58f 100644 (file)
 class MidSideDecoder : public AudioProcessor
 {
 public:
-       std::string name () const;
-       std::string id () const;
-       int out_channels () const;
-       std::shared_ptr<AudioProcessor> clone (int) const;
-       std::shared_ptr<AudioBuffers> run (std::shared_ptr<const AudioBuffers>, int channels);
-       void make_audio_mapping_default (AudioMapping& mapping) const;
-       std::vector<NamedChannel> input_names () const;
+       std::string name () const override;
+       std::string id () const override;
+       int out_channels () const override;
+       std::shared_ptr<AudioProcessor> clone (int) const override;
+       std::shared_ptr<AudioBuffers> run (std::shared_ptr<const AudioBuffers>, int channels) override;
+       void make_audio_mapping_default (AudioMapping& mapping) const override;
+       std::vector<NamedChannel> input_names () const override;
 };
index ce6017ca2f3879440addfccb55ef044ecf784783..2c7232f14ca608ec2461975500216651882cc032 100644 (file)
@@ -23,5 +23,5 @@
 class NullLog : public Log
 {
 private:
-       void do_log (std::shared_ptr<const LogEntry>) {}
+       void do_log (std::shared_ptr<const LogEntry>) override {}
 };
index c9885654bd3ae3e830c7a6f635d25d205999f8d8..1111b66c09dea38b64b46189042fa030ba7cb33a 100644 (file)
@@ -35,12 +35,12 @@ public:
        Result image (
                Image::Alignment alignment,
                boost::optional<dcp::Size> size = boost::optional<dcp::Size> ()
-               ) const;
+               ) const override;
 
-       void add_metadata (xmlpp::Node *) const;
-       void write_to_socket (std::shared_ptr<Socket>) const;
-       bool same (std::shared_ptr<const ImageProxy>) const;
-       size_t memory_used () const;
+       void add_metadata (xmlpp::Node *) const override;
+       void write_to_socket (std::shared_ptr<Socket>) const override;
+       bool same (std::shared_ptr<const ImageProxy>) const override;
+       size_t memory_used () const override;
 
 private:
        std::shared_ptr<Image> _image;
index 6103d207d9bffc0870edbd194cee96ab75d56936..84cecb80b4853991cc85b6bfb1c9e95091dceb3c 100644 (file)
@@ -62,7 +62,7 @@ public:
 
        explicit Screen (cxml::ConstNodePtr);
 
-       void as_xml (xmlpp::Element *) const;
+       void as_xml (xmlpp::Element *) const override;
        std::vector<std::string> trusted_device_thumbprints () const;
 
        std::shared_ptr<Cinema> cinema;
index 2b010665cae79e08bb57b71a2035576996e9e9da..76fb72b8a364c0866310ea77ad161d931626eb4c 100644 (file)
@@ -53,9 +53,9 @@ public:
 
        ~SendKDMEmailJob ();
 
-       std::string name () const;
-       std::string json_name () const;
-       void run ();
+       std::string name () const override;
+       std::string json_name () const override;
+       void run () override;
 
 private:
        dcp::NameFormat _container_name_format;
index 0dcc243db6499295df9c3973e4b3e1a9163ab025..2d5f30ee82a218402a21c07414cf4cf0402e53c4 100644 (file)
@@ -29,5 +29,5 @@ public:
        explicit StdoutLog (int types);
 
 private:
-       void do_log (std::shared_ptr<const LogEntry> entry);
+       void do_log (std::shared_ptr<const LogEntry> entry) override;
 };
index 143ada024be67f45c0370311de362413fc15135c..542a846460e1c5637618befcc2498f6812d67920 100644 (file)
@@ -28,7 +28,7 @@ class StringLogEntry : public LogEntry
 public:
        StringLogEntry (int type, std::string message);
 
-       std::string message () const {
+       std::string message () const override {
                return _message;
        }
 
index ef908051c623c4cbdeae3aa3be7e6811ad96e2ca..9c7d4cea0b041b8f94d296518e347a54608b8ce5 100644 (file)
@@ -42,13 +42,13 @@ public:
                return std::dynamic_pointer_cast<const StringTextFileContent> (Content::shared_from_this ());
        }
 
-       void examine (std::shared_ptr<const Film> film, std::shared_ptr<Job>);
-       std::string summary () const;
-       std::string technical_summary () const;
-       void as_xml (xmlpp::Node *, bool with_paths) const;
-       dcpomatic::DCPTime full_length (std::shared_ptr<const Film> film) const;
-       dcpomatic::DCPTime approximate_length () const;
-       std::string identifier () const;
+       void examine (std::shared_ptr<const Film> film, std::shared_ptr<Job>) override;
+       std::string summary () const override;
+       std::string technical_summary () const override;
+       void as_xml (xmlpp::Node *, bool with_paths) const override;
+       dcpomatic::DCPTime full_length (std::shared_ptr<const Film> film) const override;
+       dcpomatic::DCPTime approximate_length () const override;
+       std::string identifier () const override;
 
 private:
        dcpomatic::ContentTime _length;
index aa8c80e6e9152341d2f114a0118e4934dcf3fc4e..5887f34c61f81fed1bdfaa9fc923273b4c122392 100644 (file)
@@ -31,10 +31,10 @@ class StringTextFileDecoder : public Decoder, public StringTextFile
 public:
        StringTextFileDecoder (std::shared_ptr<const Film> film, std::shared_ptr<const StringTextFileContent>);
 
-       void seek (dcpomatic::ContentTime time, bool accurate);
-       bool pass ();
+       void seek (dcpomatic::ContentTime time, bool accurate) override;
+       bool pass () override;
 
-       std::vector<dcpomatic::FontData> fonts () const;
+       std::vector<dcpomatic::FontData> fonts () const override;
 
 private:
        dcpomatic::ContentTimePeriod content_time_period (sub::Subtitle s) const;
index d1355afb02c4b7eaf135917e5dccc487641d7947..39d6e828d97864552e5833b9dad9bae3028aa363 100644 (file)
@@ -48,7 +48,7 @@ public:
                dcpomatic::ContentTime first
                );
 
-       boost::optional<dcpomatic::ContentTime> position (std::shared_ptr<const Film>) const {
+       boost::optional<dcpomatic::ContentTime> position (std::shared_ptr<const Film>) const override {
                return _position;
        }
 
@@ -60,7 +60,7 @@ public:
        void emit_plain (dcpomatic::ContentTimePeriod period, sub::Subtitle const & subtitle);
        void emit_stop (dcpomatic::ContentTime to);
 
-       void seek ();
+       void seek () override;
 
        std::shared_ptr<const TextContent> content () const {
                return _content;
index afd5ebda431488d0bac516061ab23b59860c0c6e..a3da164bf56f8ea6990af4204f4c357332123e32 100644 (file)
@@ -33,10 +33,10 @@ public:
        explicit UploadJob (std::shared_ptr<const Film>);
        ~UploadJob ();
 
-       std::string name () const;
-       std::string json_name () const;
-       void run ();
-       std::string status () const;
+       std::string name () const override;
+       std::string json_name () const override;
+       void run () override;
+       std::string status () const override;
 
 private:
        void set_status (std::string);
index b357616ef0038e758b8ac0d6a17fccb1e4fe0c75..9e364dcc9611fb2f2df3ac4f28703f970da53aa1 100644 (file)
@@ -36,14 +36,14 @@ class UpmixerA : public AudioProcessor
 public:
        explicit UpmixerA (int sampling_rate);
 
-       std::string name () const;
-       std::string id () const;
-       int out_channels () const;
-       std::shared_ptr<AudioProcessor> clone (int) const;
-       std::shared_ptr<AudioBuffers> run (std::shared_ptr<const AudioBuffers>, int channels);
-       void flush ();
-       void make_audio_mapping_default (AudioMapping& mapping) const;
-       std::vector<NamedChannel> input_names () const;
+       std::string name () const override;
+       std::string id () const override;
+       int out_channels () const override;
+       std::shared_ptr<AudioProcessor> clone (int) const override;
+       std::shared_ptr<AudioBuffers> run (std::shared_ptr<const AudioBuffers>, int channels) override;
+       void flush () override;
+       void make_audio_mapping_default (AudioMapping& mapping) const override;
+       std::vector<NamedChannel> input_names () const override;
 
 private:
        BandPassAudioFilter _left;
index fc30e2a28e88bc7e08c3a082b23b8df9c926db99..611ed6a90fbdb61de4dd2318d5d0f3f5ed6ea521 100644 (file)
@@ -34,14 +34,14 @@ class UpmixerB : public AudioProcessor
 public:
        explicit UpmixerB (int sampling_rate);
 
-       std::string name () const;
-       std::string id () const;
-       int out_channels () const;
-       std::shared_ptr<AudioProcessor> clone (int) const;
-       std::shared_ptr<AudioBuffers> run (std::shared_ptr<const AudioBuffers>, int channels);
-       void flush ();
-       void make_audio_mapping_default (AudioMapping& mapping) const;
-       std::vector<NamedChannel> input_names () const;
+       std::string name () const override;
+       std::string id () const override;
+       int out_channels () const override;
+       std::shared_ptr<AudioProcessor> clone (int) const override;
+       std::shared_ptr<AudioBuffers> run (std::shared_ptr<const AudioBuffers>, int channels) override;
+       void flush () override;
+       void make_audio_mapping_default (AudioMapping& mapping) const override;
+       std::vector<NamedChannel> input_names () const override;
 
 private:
        LowPassAudioFilter _lfe;
index 9d69f743a33d6c08d3efae5af36329a4278cb709..828ac66a20af1cbbff7785f039b9c1fd8a32bddb 100644 (file)
@@ -56,11 +56,11 @@ public:
        friend struct ffmpeg_pts_offset_test;
        friend void ffmpeg_decoder_sequential_test_one (boost::filesystem::path file, float fps, int gaps, int video_length);
 
-       boost::optional<dcpomatic::ContentTime> position (std::shared_ptr<const Film>) const {
+       boost::optional<dcpomatic::ContentTime> position (std::shared_ptr<const Film>) const override {
                return _position;
        }
 
-       void seek ();
+       void seek () override;
        void emit (std::shared_ptr<const Film> film, std::shared_ptr<const ImageProxy>, Frame frame);
 
        boost::signals2::signal<void (ContentVideo)> Data;
index 4d494bb632291d1d80b0d30e87f5dd350c7e00c9..42fc421e8d8316ac780acafd6ee1574c4200a226 100644 (file)
@@ -1585,7 +1585,7 @@ public:
 
 private:
 
-       bool OnInit ()
+       bool OnInit () override
        {
                try {
                        wxInitAllImageHandlers ();
@@ -1693,13 +1693,13 @@ private:
                return true;
        }
 
-       void OnInitCmdLine (wxCmdLineParser& parser)
+       void OnInitCmdLine (wxCmdLineParser& parser) override
        {
                parser.SetDesc (command_line_description);
                parser.SetSwitchChars (wxT ("-"));
        }
 
-       bool OnCmdLineParsed (wxCmdLineParser& parser)
+       bool OnCmdLineParsed (wxCmdLineParser& parser) override
        {
                if (parser.Found (wxT("version"))) {
                        cout << "dcpomatic version " << dcpomatic_version << " " << dcpomatic_git_commit << "\n";
@@ -1769,14 +1769,14 @@ private:
        }
 
        /* An unhandled exception has occurred inside the main event loop */
-       bool OnExceptionInMainLoop ()
+       bool OnExceptionInMainLoop () override
        {
                report_exception ();
                /* This will terminate the program */
                return false;
        }
 
-       void OnUnhandledException ()
+       void OnUnhandledException () override
        {
                report_exception ();
        }
index 29f79ff080bf840276773c1159c12f3078c21fc1..d3aa2500b6fe22965c206e8dae2406df308059b0 100644 (file)
@@ -369,7 +369,7 @@ public:
                , _frame (frame)
        {}
 
-       void handle (shared_ptr<Socket> socket)
+       void handle (shared_ptr<Socket> socket) override
        {
                try {
                        int const length = socket->read_uint32 ();
@@ -390,7 +390,7 @@ private:
 
 class App : public wxApp
 {
-       bool OnInit ()
+       bool OnInit () override
        {
                wxInitAllImageHandlers ();
 
@@ -469,13 +469,13 @@ class App : public wxApp
                signal_manager->ui_idle ();
        }
 
-       void OnInitCmdLine (wxCmdLineParser& parser)
+       void OnInitCmdLine (wxCmdLineParser& parser) override
        {
                parser.SetDesc (command_line_description);
                parser.SetSwitchChars (wxT ("-"));
        }
 
-       bool OnCmdLineParsed (wxCmdLineParser& parser)
+       bool OnCmdLineParsed (wxCmdLineParser& parser) override
        {
                for (size_t i = 0; i < parser.GetParamCount(); ++i) {
                        films_to_load.push_back (wx_to_std(parser.GetParam(i)));
index ec5d8e782db122dc4a15814432aa3ddae5bb2f2c..1907a604522149e1b642ee84022d4ef2fea6ccb2 100644 (file)
@@ -418,7 +418,7 @@ public:
                : _frame (nullptr)
        {}
 
-       bool OnInit ()
+       bool OnInit () override
        {
                try {
                        Config::FailedToLoad.connect (boost::bind (&App::config_failed_to_load, this));
@@ -489,13 +489,13 @@ public:
                return true;
        }
 
-       void OnInitCmdLine (wxCmdLineParser& parser)
+       void OnInitCmdLine (wxCmdLineParser& parser) override
        {
                parser.SetDesc (command_line_description);
                parser.SetSwitchChars (wxT ("-"));
        }
 
-       bool OnCmdLineParsed (wxCmdLineParser& parser)
+       bool OnCmdLineParsed (wxCmdLineParser& parser) override
        {
                _skip_alpha_check = parser.Found(wxT("sure"));
 
@@ -549,14 +549,14 @@ public:
                }
        }
 
-       bool OnExceptionInMainLoop ()
+       bool OnExceptionInMainLoop () override
        {
                report_exception ();
                /* This will terminate the program */
                return false;
        }
 
-       void OnUnhandledException ()
+       void OnUnhandledException () override
        {
                report_exception ();
        }
index 24bd3b4d7f776ad60f369fb1e16cc80b4e2b52a3..481b0fb8b0130d936fc1c2aeb20400296e48a0a8 100644 (file)
@@ -620,7 +620,7 @@ public:
 
 private:
 
-       bool OnInit ()
+       bool OnInit () override
        {
                wxSplashScreen* splash = nullptr;
 
@@ -719,7 +719,7 @@ private:
                return false;
        }
 
-       void OnUnhandledException ()
+       void OnUnhandledException () override
        {
                error_dialog (nullptr, _("An unknown exception occurred.") + "  " + REPORT_PROBLEM);
        }
index 8c9f016d4b71a65f16b8a871a0b74b5befd1423e..b83d9c482970daaaee58240687b4470dae0abbfe 100644 (file)
@@ -1023,7 +1023,7 @@ public:
                , _frame (frame)
        {}
 
-       void handle (shared_ptr<Socket> socket)
+       void handle (shared_ptr<Socket> socket) override
        {
                try {
                        int const length = socket->read_uint32 ();
@@ -1057,7 +1057,7 @@ public:
 
 private:
 
-       bool OnInit ()
+       bool OnInit () override
        {
                wxSplashScreen* splash = nullptr;
                try {
@@ -1156,13 +1156,13 @@ private:
                return true;
        }
 
-       void OnInitCmdLine (wxCmdLineParser& parser)
+       void OnInitCmdLine (wxCmdLineParser& parser) override
        {
                parser.SetDesc (command_line_description);
                parser.SetSwitchChars (wxT ("-"));
        }
 
-       bool OnCmdLineParsed (wxCmdLineParser& parser)
+       bool OnCmdLineParsed (wxCmdLineParser& parser) override
        {
                if (parser.GetParamCount() > 0) {
                        _dcp_to_load = wx_to_std (parser.GetParam (0));
@@ -1207,14 +1207,14 @@ private:
        }
 
        /* An unhandled exception has occurred inside the main event loop */
-       bool OnExceptionInMainLoop ()
+       bool OnExceptionInMainLoop () override
        {
                report_exception ();
                /* This will terminate the program */
                return false;
        }
 
-       void OnUnhandledException ()
+       void OnUnhandledException () override
        {
                report_exception ();
        }
index 6f06c0624e91109475a6b320c4ca5e8a56ed6fa1..dbc5e81ddf673b445876b42863260daddfdd936e 100644 (file)
@@ -85,7 +85,7 @@ public:
                return _content_view->selected ();
        }
 
-       shared_ptr<Content> get (string digest) const
+       shared_ptr<Content> get (string digest) const override
        {
                return _content_view->get (digest);
        }
@@ -590,7 +590,7 @@ public:
 
 private:
 
-       bool OnInit ()
+       bool OnInit () override
        try
        {
                wxInitAllImageHandlers ();
@@ -645,7 +645,7 @@ private:
        }
 
        /* An unhandled exception has occurred inside the main event loop */
-       bool OnExceptionInMainLoop ()
+       bool OnExceptionInMainLoop () override
        {
                try {
                        throw;
@@ -674,7 +674,7 @@ private:
                return false;
        }
 
-       void OnUnhandledException ()
+       void OnUnhandledException () override
        {
                error_dialog (0, _("An unknown exception occurred.") + "  " + REPORT_PROBLEM);
        }
index 401fb1077d047c83921f329d1570f8413f6fe3ae..de25f531d733a6e8fe786dfe8b21754eb7185280 100644 (file)
@@ -87,7 +87,7 @@ public:
        boost::signals2::signal<void(int)> Removed;
 
 private:
-       void do_log (shared_ptr<const LogEntry> entry)
+       void do_log (shared_ptr<const LogEntry> entry) override
        {
                time_t const s = entry->seconds ();
                struct tm* local = localtime (&s);
@@ -226,7 +226,7 @@ public:
                Bind (wxEVT_MENU, boost::bind (&TaskBarIcon::quit, this), ID_quit);
        }
 
-       wxMenu* CreatePopupMenu ()
+       wxMenu* CreatePopupMenu () override
        {
                auto menu = new wxMenu;
                menu->Append (ID_status, std_to_wx ("Status..."));
@@ -273,7 +273,7 @@ public:
 
 private:
 
-       bool OnInit ()
+       bool OnInit () override
        {
                if (!wxApp::OnInit()) {
                        return false;
@@ -327,7 +327,7 @@ private:
                return true;
        }
 
-       int OnExit ()
+       int OnExit () override
        {
                delete _icon;
                return wxApp::OnExit ();
index 12dd0cadf287dff0859945e12c80afdcd1cdce49..8ba69ec79bdee838c23da4c2035d739afc64b5a5 100644 (file)
@@ -36,7 +36,7 @@ class AudioDialog : public wxDialog
 public:
        AudioDialog (wxWindow* parent, std::shared_ptr<Film> film, std::weak_ptr<FilmViewer> viewer, std::shared_ptr<Content> content = std::shared_ptr<Content>());
 
-       bool Show (bool show = true);
+       bool Show (bool show = true) override;
 
        void set_cursor (boost::optional<dcpomatic::DCPTime> time, boost::optional<float> db);
 
index fce7d3f5d6543aa3387c48d50b519918a2026a92..addae042142d5ecc63e6e15534cc825de1288e7f 100644 (file)
@@ -25,7 +25,7 @@ class BarcoAlchemyCertificatePanel : public CredentialsDownloadCertificatePanel
 public:
        BarcoAlchemyCertificatePanel (DownloadCertificateDialog* dialog);
 
-       bool ready_to_download () const;
-       void do_download ();
-       wxString name () const;
+       bool ready_to_download () const override;
+       void do_download () override;
+       wxString name () const override;
 };
index 6d9ccba31cc3bbfb96cb3f8d5624945bb03c9bcb..91dca82e9fa6feb379ea2527f355a8fa6683ea39 100644 (file)
@@ -28,10 +28,10 @@ public:
        BatchJobView (std::shared_ptr<Job> job, wxWindow* parent, wxWindow* container, wxFlexGridSizer* table);
 
 private:
-       int insert_position () const;
-       void job_list_changed ();
+       int insert_position () const override;
+       void job_list_changed () override;
 
-       void finish_setup (wxWindow* parent, wxSizer* sizer);
+       void finish_setup (wxWindow* parent, wxSizer* sizer) override;
        void higher_priority_clicked ();
        void lower_priority_clicked ();
 
index 8e6789e448311346f7ddb838c7e353979e9fa80e..5176cd2e8cce4d3245e58a2d13eb3ce8aa1f4fe2 100644 (file)
@@ -32,8 +32,8 @@ class CheckBox : public wxCheckBox, public I18NHook
 public:
        CheckBox (wxWindow* parent, wxString label);
 
-       void set_text (wxString text);
-       wxString get_text () const;
+       void set_text (wxString text) override;
+       wxString get_text () const override;
 };
 
 
index 41278cb7c6c3d83e5c3590647d1b771cbb15eba9..54536a81c3cab656360c0650edc7340c03522b32 100644 (file)
@@ -25,6 +25,6 @@ class ChristieCertificatePanel : public CredentialsDownloadCertificatePanel
 public:
        ChristieCertificatePanel (DownloadCertificateDialog* dialog);
 
-       void do_download ();
-       wxString name () const;
+       void do_download () override;
+       wxString name () const override;
 };
index b7f3a269f9ac442e1cbe887cf30234da24c18daf..ce5df2c5b84b43badb59a4fbcee7cd77c8c3729f 100644 (file)
@@ -54,7 +54,7 @@ public:
        Page (wxSize panel_size, int border);
        virtual ~Page () {}
 
-       wxWindow* CreateWindow (wxWindow* parent);
+       wxWindow* CreateWindow (wxWindow* parent) override;
 
 protected:
        wxWindow* create_window (wxWindow* parent);
@@ -79,10 +79,10 @@ class GeneralPage : public Page
 public:
        GeneralPage (wxSize panel_size, int border);
 
-       wxString GetName () const;
+       wxString GetName () const override;
 
 #ifdef DCPOMATIC_OSX
-       wxBitmap GetLargeIcon () const
+       wxBitmap GetLargeIcon () const override
        {
                return wxBitmap(bitmap_path("general"), wxBITMAP_TYPE_PNG);
        }
@@ -91,7 +91,7 @@ public:
 protected:
        void add_language_controls (wxGridBagSizer* table, int& r);
        void add_update_controls (wxGridBagSizer* table, int& r);
-       virtual void config_changed ();
+       void config_changed () override;
 
 private:
        void setup_sensitivity ();
@@ -156,10 +156,10 @@ public:
                : Page (panel_size, border)
        {}
 
-       wxString GetName () const;
+       wxString GetName () const override;
 
 #ifdef DCPOMATIC_OSX
-       wxBitmap GetLargeIcon () const
+       wxBitmap GetLargeIcon () const override
        {
                return wxBitmap(bitmap_path("keys"), wxBITMAP_TYPE_PNG);
        }
@@ -167,10 +167,10 @@ public:
 
 private:
 
-       void setup ();
+       void setup () override;
 
        void export_decryption_certificate ();
-       void config_changed () {}
+       void config_changed () override {}
        bool nag_alter_decryption_chain ();
        void decryption_advanced ();
        void signing_advanced ();
@@ -187,10 +187,10 @@ public:
                : Page (panel_size, border)
        {}
 
-       wxString GetName() const;
+       wxString GetName() const override;
 
 #ifdef DCPOMATIC_OSX
-       wxBitmap GetLargeIcon () const
+       wxBitmap GetLargeIcon () const override
        {
                return wxBitmap(bitmap_path("sound"), wxBITMAP_TYPE_PNG);
        }
@@ -198,8 +198,8 @@ public:
 
 private:
 
-       void setup ();
-       void config_changed ();
+       void setup () override;
+       void config_changed () override;
         boost::optional<std::string> get_sound_output ();
        void sound_changed ();
        void sound_output_changed ();
@@ -219,15 +219,15 @@ class LocationsPage : public Page
 public:
        LocationsPage (wxSize panel_size, int border);
 
-       wxString GetName () const;
+       wxString GetName () const override;
 
 #ifdef DCPOMATIC_OSX
-       wxBitmap GetLargeIcon () const;
+       wxBitmap GetLargeIcon () const override;
 #endif
 
 private:
-       void setup ();
-       void config_changed ();
+       void setup () override;
+       void config_changed () override;
        void content_directory_changed ();
        void playlist_directory_changed ();
        void kdm_directory_changed ();
index e25dedceabc94064de40e7386361cfed4d428b41..650b7966838abc828d9c99a54e0a8ac872c1b474 100644 (file)
@@ -48,7 +48,7 @@ class LimitedSplitter : public wxSplitterWindow
 public:
        LimitedSplitter (wxWindow* parent);
 
-       bool OnSashPositionChange (int new_position)
+       bool OnSashPositionChange (int new_position) override
        {
                /* Try to stop the top bit of the splitter getting so small that buttons disappear */
                return new_position > 220;
index 1ad7c541ae9645ae8b9109e586be401437967631..d7c1a43ed7b971307a5fc1b5810e07e7945c4fb4 100644 (file)
@@ -39,7 +39,7 @@ public:
        std::shared_ptr<Content> selected () const;
        void update ();
 
-       std::shared_ptr<Content> get (std::string digest) const;
+       std::shared_ptr<Content> get (std::string digest) const override;
 
 private:
        void add (std::shared_ptr<Content> content);
index b802768e924bd5cfed44d8553819016bc1076e6f..b9af14f5b27a61d87fe6cb1de90083f2faaffac3 100644 (file)
@@ -38,7 +38,7 @@ public:
                        std::function<void ()> unset_password
                        );
 
-       virtual bool ready_to_download () const;
+       bool ready_to_download () const override;
 
 private:
        void username_changed ();
index 682fa5fe67e91ae783f0e714f4742322917d0d4e..aba41b12562908ecadc3bda3a283183e7ec1ddb8 100644 (file)
@@ -29,8 +29,8 @@ class Button : public wxButton, public I18NHook
 public:
        Button (wxWindow* parent, wxString label, wxPoint pos = wxDefaultPosition, wxSize = wxDefaultSize, long style = 0);
 
-       void set_text (wxString text);
-       wxString get_text () const;
+       void set_text (wxString text) override;
+       wxString get_text () const override;
 };
 
 #endif
index 1c14ad35ca88553cb6c83fbbb9a89bfbf8c1d1ba..0699bff223f4808b541c84d1e5ef59eccbc82d70 100644 (file)
@@ -89,7 +89,7 @@ public:
        {}
 
 private:
-       void setup ()
+       void setup () override
        {
                auto table = new wxGridBagSizer (DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
                _panel->GetSizer()->Add (table, 1, wxALL | wxEXPAND, _border);
@@ -146,7 +146,7 @@ private:
                _automatic_audio_analysis->Bind (wxEVT_CHECKBOX, boost::bind (&FullGeneralPage::automatic_audio_analysis_changed, this));
        }
 
-       void config_changed ()
+       void config_changed () override
        {
                auto config = Config::instance ();
 
@@ -246,20 +246,20 @@ public:
                : Page (panel_size, border)
        {}
 
-       wxString GetName () const
+       wxString GetName () const override
        {
                return _("Defaults");
        }
 
 #ifdef DCPOMATIC_OSX
-       wxBitmap GetLargeIcon () const
+       wxBitmap GetLargeIcon () const override
        {
                return wxBitmap(bitmap_path("defaults"), wxBITMAP_TYPE_PNG);
        }
 #endif
 
 private:
-       void setup ()
+       void setup () override
        {
                auto table = new wxFlexGridSizer (2, DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
                table->AddGrowableCol (1, 1);
@@ -385,7 +385,7 @@ private:
                }
        }
 
-       void config_changed ()
+       void config_changed () override
        {
                auto config = Config::instance ();
 
@@ -545,20 +545,20 @@ public:
                : Page (panel_size, border)
        {}
 
-       wxString GetName () const
+       wxString GetName () const override
        {
                return _("Servers");
        }
 
 #ifdef DCPOMATIC_OSX
-       wxBitmap GetLargeIcon () const
+       wxBitmap GetLargeIcon () const override
        {
                return wxBitmap(bitmap_path("servers"), wxBITMAP_TYPE_PNG);
        }
 #endif
 
 private:
-       void setup ()
+       void setup () override
        {
                _use_any_servers = new CheckBox (_panel, _("Search network for servers"));
                _panel->GetSizer()->Add (_use_any_servers, 0, wxALL, _border);
@@ -578,7 +578,7 @@ private:
                _use_any_servers->Bind (wxEVT_CHECKBOX, boost::bind(&EncodingServersPage::use_any_servers_changed, this));
        }
 
-       void config_changed ()
+       void config_changed () override
        {
                checked_set (_use_any_servers, Config::instance()->use_any_servers ());
                _servers_list->refresh ();
@@ -606,20 +606,20 @@ public:
                : Page (panel_size, border)
        {}
 
-       wxString GetName () const
+       wxString GetName () const override
        {
                return _("TMS");
        }
 
 #ifdef DCPOMATIC_OSX
-       wxBitmap GetLargeIcon () const
+       wxBitmap GetLargeIcon () const override
        {
                return wxBitmap(bitmap_path("tms"), wxBITMAP_TYPE_PNG);
        }
 #endif
 
 private:
-       void setup ()
+       void setup () override
        {
                _upload = new CheckBox (_panel, _("Upload DCP to TMS after creation"));
                _panel->GetSizer()->Add (_upload, 0, wxALL | wxEXPAND, _border);
@@ -659,7 +659,7 @@ private:
                _tms_password->Changed.connect (boost::bind (&TMSPage::tms_password_changed, this));
        }
 
-       void config_changed ()
+       void config_changed () override
        {
                auto config = Config::instance ();
 
@@ -717,20 +717,20 @@ public:
                : Page (panel_size, border)
        {}
 
-       wxString GetName () const
+       wxString GetName () const override
        {
                return _("Email");
        }
 
 #ifdef DCPOMATIC_OSX
-       wxBitmap GetLargeIcon () const
+       wxBitmap GetLargeIcon () const override
        {
                return wxBitmap(bitmap_path("email"), wxBITMAP_TYPE_PNG);
        }
 #endif
 
 private:
-       void setup ()
+       void setup () override
        {
                auto table = new wxFlexGridSizer (2, DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
                table->AddGrowableCol (1, 1);
@@ -776,7 +776,7 @@ private:
                _send_test_email->Bind (wxEVT_BUTTON, boost::bind(&EmailPage::send_test_email_clicked, this));
        }
 
-       void config_changed ()
+       void config_changed () override
        {
                auto config = Config::instance ();
 
@@ -889,20 +889,20 @@ public:
 #endif
        {}
 
-       wxString GetName () const
+       wxString GetName () const override
        {
                return _("KDM Email");
        }
 
 #ifdef DCPOMATIC_OSX
-       wxBitmap GetLargeIcon () const
+       wxBitmap GetLargeIcon () const override
        {
                return wxBitmap(bitmap_path("kdm_email"), wxBITMAP_TYPE_PNG);
        }
 #endif
 
 private:
-       void setup ()
+       void setup () override
        {
                auto table = new wxFlexGridSizer (2, DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
                table->AddGrowableCol (1, 1);
@@ -948,9 +948,9 @@ private:
                _reset_email->Bind (wxEVT_BUTTON, boost::bind (&KDMEmailPage::reset_email, this));
        }
 
-       void config_changed ()
+       void config_changed () override
        {
-               Config* config = Config::instance ();
+               auto config = Config::instance ();
 
                checked_set (_subject, config->kdm_subject ());
                checked_set (_from, config->kdm_from ());
@@ -1011,20 +1011,20 @@ public:
 #endif
        {}
 
-       wxString GetName () const
+       wxString GetName () const override
        {
                return _("Notifications");
        }
 
 #ifdef DCPOMATIC_OSX
-       wxBitmap GetLargeIcon () const
+       wxBitmap GetLargeIcon () const override
        {
                return wxBitmap(bitmap_path("notifications"), wxBITMAP_TYPE_PNG);
        }
 #endif
 
 private:
-       void setup ()
+       void setup () override
        {
                auto table = new wxFlexGridSizer (2, DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
                table->AddGrowableCol (1, 1);
@@ -1100,7 +1100,7 @@ private:
                _reset_email->Enable(s);
        }
 
-       void config_changed ()
+       void config_changed () override
        {
                auto config = Config::instance ();
 
@@ -1184,20 +1184,20 @@ public:
 #endif
        {}
 
-       wxString GetName () const
+       wxString GetName () const override
        {
                return _("Cover Sheet");
        }
 
 #ifdef DCPOMATIC_OSX
-       wxBitmap GetLargeIcon () const
+       wxBitmap GetLargeIcon () const override
        {
                return wxBitmap(bitmap_path("cover_sheet"), wxBITMAP_TYPE_PNG);
        }
 #endif
 
 private:
-       void setup ()
+       void setup () override
        {
                _cover_sheet = new wxTextCtrl (_panel, wxID_ANY, wxEmptyString, wxDefaultPosition, wxSize (-1, 200), wxTE_MULTILINE);
                _panel->GetSizer()->Add (_cover_sheet, 0, wxEXPAND | wxALL, _border);
@@ -1209,7 +1209,7 @@ private:
                _reset_cover_sheet->Bind (wxEVT_BUTTON, boost::bind (&CoverSheetPage::reset_cover_sheet, this));
        }
 
-       void config_changed ()
+       void config_changed () override
        {
                checked_set (_cover_sheet, Config::instance()->cover_sheet());
        }
@@ -1243,20 +1243,20 @@ public:
                : Page (panel_size, border)
        {}
 
-       wxString GetName () const
+       wxString GetName () const override
        {
                return _("Identifiers");
        }
 
 #ifdef DCPOMATIC_OSX
-       wxBitmap GetLargeIcon () const
+       wxBitmap GetLargeIcon () const override
        {
                return wxBitmap(bitmap_path("identifiers"), wxBITMAP_TYPE_PNG);
        }
 #endif
 
 private:
-       void setup ()
+       void setup () override
        {
                auto table = new wxFlexGridSizer (2, DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
                table->AddGrowableCol (1, 1);
@@ -1301,7 +1301,7 @@ private:
                _j2k_comment->Bind (wxEVT_TEXT, boost::bind(&IdentifiersPage::j2k_comment_changed, this));
        }
 
-       void config_changed ()
+       void config_changed () override
        {
                auto config = Config::instance ();
                checked_set (_issuer, config->dcp_issuer ());
@@ -1361,13 +1361,13 @@ public:
                : Page (panel_size, border)
        {}
 
-       wxString GetName () const
+       wxString GetName () const override
        {
                return _("Advanced");
        }
 
 #ifdef DCPOMATIC_OSX
-       wxBitmap GetLargeIcon () const
+       wxBitmap GetLargeIcon () const override
        {
                return wxBitmap(bitmap_path("advanced"), wxBITMAP_TYPE_PNG);
        }
@@ -1385,7 +1385,7 @@ private:
                table->Add (m, 0, flags, DCPOMATIC_SIZER_Y_GAP);
        }
 
-       void setup ()
+       void setup () override
        {
                auto table = new wxFlexGridSizer (2, DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
                table->AddGrowableCol (1, 1);
@@ -1545,7 +1545,7 @@ private:
 #endif
        }
 
-       void config_changed ()
+       void config_changed () override
        {
                auto config = Config::instance ();
 
index a26dcc0bb81a2862ca77a2d612292e6d37effa1f..a6d5e49a65de69ef0201a26171936ce6ac2d6b05 100644 (file)
@@ -111,7 +111,7 @@ public:
        }
 
 private:
-       wxString OnGetItemText (long item, long column) const
+       wxString OnGetItemText (long item, long column) const override
        {
                if (column == 0) {
                        return _matching_subtags[item].subtag;
index b69c7a51fdab3fd7ca74284540428d352e8830db..e41254e4512750dadb102b04581e7da8f2f06953 100644 (file)
@@ -25,6 +25,6 @@ class GDCCertificatePanel : public CredentialsDownloadCertificatePanel
 public:
        GDCCertificatePanel (DownloadCertificateDialog* dialog);
 
-       void do_download ();
-       wxString name () const;
+       void do_download () override;
+       wxString name () const override;
 };
index acd8111b6e6c95b48ee5f4de358bc0662b9de0ee..7d4f72ad55d5d13612b554060ab14f76dcbbfaa1 100644 (file)
@@ -33,10 +33,10 @@ public:
        void setup ();
 
 private:
-       int insert_position () const;
-       void finish_setup (wxWindow* parent, wxSizer* sizer);
+       int insert_position () const override;
+       void finish_setup (wxWindow* parent, wxSizer* sizer) override;
        void pause_clicked ();
-       void finished ();
+       void finished () override;
 
        wxButton* _pause = nullptr;
 };
index 91434beaeb6a64b36539f3acf2d74c9faeb72757..af0f8641efe9c23a91d34fcabc9604284dcf8890 100644 (file)
@@ -78,7 +78,7 @@ public:
        {}
 
 private:
-       void setup ()
+       void setup () override
        {
                wxGridBagSizer* table = new wxGridBagSizer (DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
                _panel->GetSizer()->Add (table, 1, wxALL | wxEXPAND, _border);
@@ -132,7 +132,7 @@ private:
                _debug_log_file->Bind (wxEVT_FILEPICKER_CHANGED, bind(&PlayerGeneralPage::debug_log_file_changed, this));
        }
 
-       void config_changed ()
+       void config_changed () override
        {
                GeneralPage::config_changed ();
 
@@ -228,13 +228,13 @@ public:
                , _log_timing (0)
        {}
 
-       wxString GetName () const
+       wxString GetName () const override
        {
                return _("Advanced");
        }
 
 #ifdef DCPOMATIC_OSX
-       wxBitmap GetLargeIcon () const
+       wxBitmap GetLargeIcon () const override
        {
                return wxBitmap ("advanced", wxBITMAP_TYPE_PNG_RESOURCE);
        }
@@ -252,9 +252,9 @@ private:
                table->Add (m, 0, flags, DCPOMATIC_SIZER_Y_GAP);
        }
 
-       void setup ()
+       void setup () override
        {
-               wxFlexGridSizer* table = new wxFlexGridSizer (2, DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
+               auto table = new wxFlexGridSizer (2, DCPOMATIC_SIZER_X_GAP, DCPOMATIC_SIZER_Y_GAP);
                table->AddGrowableCol (1, 1);
                _panel->GetSizer()->Add (table, 1, wxALL | wxEXPAND, _border);
 
@@ -288,9 +288,9 @@ private:
 #endif
        }
 
-       void config_changed ()
+       void config_changed () override
        {
-               Config* config = Config::instance ();
+               auto config = Config::instance ();
 
                checked_set (_log_general, config->log_types() & LogEntry::TYPE_GENERAL);
                checked_set (_log_warning, config->log_types() & LogEntry::TYPE_WARNING);
@@ -339,17 +339,17 @@ private:
 wxPreferencesEditor*
 create_player_config_dialog ()
 {
-       wxPreferencesEditor* e = new wxPreferencesEditor (_("DCP-o-matic Player Preferences"));
+       auto e = new wxPreferencesEditor (_("DCP-o-matic Player Preferences"));
 
 #ifdef DCPOMATIC_OSX
        /* Width that we force some of the config panels to be on OSX so that
           the containing window doesn't shrink too much when we select those panels.
           This is obviously an unpleasant hack.
        */
-       wxSize ps = wxSize (520, -1);
+       auto ps = wxSize (520, -1);
        int const border = 16;
 #else
-       wxSize ps = wxSize (-1, -1);
+       auto ps = wxSize (-1, -1);
        int const border = 8;
 #endif
 
index 8375f952e29ebd6e12b4f11060923e4a6fa79663..57a84c128671187e35ff07f24cc4e08a8905fca1 100644 (file)
@@ -28,7 +28,7 @@ class PlaylistControls : public Controls
 public:
        PlaylistControls (wxWindow* parent, std::shared_ptr<FilmViewer> viewer);
 
-       void set_film (std::shared_ptr<Film> film);
+       void set_film (std::shared_ptr<Film> film) override;
 
        /** This is so that we can tell our parent player to reset the film
            when we have created one from a SPL.  We could call a method
@@ -36,8 +36,8 @@ public:
        */
        boost::signals2::signal<void (std::weak_ptr<Film>)> ResetFilm;
 
-       void play ();
-       void stop ();
+       void play () override;
+       void stop () override;
 
 private:
        void play_clicked ();
@@ -50,10 +50,10 @@ private:
        void update_playlist_directory ();
        void spl_selection_changed ();
        void select_playlist (int selected, int position);
-       void started ();
-       void stopped ();
-       void setup_sensitivity ();
-       void config_changed (int);
+       void started () override;
+       void stopped () override;
+       void setup_sensitivity () override;
+       void config_changed (int) override;
        void viewer_finished ();
        void reset_film ();
        void update_current_content ();
index 191924509b1a09aac1d45e23ebcd4403deb30a7d..4e0e836e86cf14be571ddc6153f0c0414eaec7cd 100644 (file)
@@ -27,15 +27,15 @@ class StandardControls : public Controls
 public:
        StandardControls (wxWindow* parent, std::shared_ptr<FilmViewer> viewer, bool editor_controls);
 
-       void play ();
-       void stop ();
+       void play () override;
+       void stop () override;
 
 private:
        void check_play_state ();
        void play_clicked ();
-       void started ();
-       void stopped ();
-       void setup_sensitivity ();
+       void started () override;
+       void stopped () override;
+       void setup_sensitivity () override;
 
        wxToggleButton* _play_button;
 };
index 0e49379a97c4cd04e34e3083d0b4f8dcf747e336..325e1f6d807ffa1a0708c5281e813b7ea0cb9d0d 100644 (file)
@@ -26,6 +26,6 @@ class StaticText : public wxStaticText, public I18NHook
 public:
        StaticText (wxWindow* parent, wxString label, wxPoint pos = wxDefaultPosition, wxSize = wxDefaultSize, long style = 0);
 
-       void set_text (wxString text);
-       wxString get_text () const;
+       void set_text (wxString text) override;
+       wxString get_text () const override;
 };
index 53b0ce2b8886214128ce476b1de2c2efe6f1f05d..2d056a22c78739842847c918fa7e4f3e57732cc0 100644 (file)
@@ -44,8 +44,8 @@ class TimePickerSpin : public TimePicker
 public:
        TimePickerSpin (wxWindow* parent, wxDateTime time);
 
-       int hours () const;
-       int minutes () const;
+       int hours () const override;
+       int minutes () const override;
 
 private:
        void changed ();
@@ -60,8 +60,8 @@ class TimePickerText : public TimePicker
 public:
        TimePickerText (wxWindow* parent, wxDateTime time);
 
-       int hours () const;
-       int minutes () const;
+       int hours () const override;
+       int minutes () const override;
 
 private:
        void changed ();
index 98aea6316accce2b5ff12656db1faf491be94c80..046f5b3e66ca9d5a288a53369bb81a0e49c28068 100644 (file)
@@ -32,9 +32,9 @@ public:
        TimelineTextContentView (Timeline& tl, std::shared_ptr<Content>, std::shared_ptr<TextContent>);
 
 private:
-       bool active () const;
-       wxColour background_colour () const;
-       wxColour foreground_colour () const;
+       bool active () const override;
+       wxColour background_colour () const override;
+       wxColour foreground_colour () const override;
 
        std::shared_ptr<TextContent> _caption;
 };
index 100228ae8b634f124a9d16ca2ca8d5950fa9dea2..4c8e090fe7f5850a376e3eabb9646f61053f661d 100644 (file)
@@ -25,11 +25,11 @@ class TimelineTimeAxisView : public TimelineView
 public:
        TimelineTimeAxisView (Timeline& tl, int y);
 
-       dcpomatic::Rect<int> bbox () const;
+       dcpomatic::Rect<int> bbox () const override;
        void set_y (int y);
 
 private:
-       void do_paint (wxGraphicsContext* gc, std::list<dcpomatic::Rect<int>> overlaps);
+       void do_paint (wxGraphicsContext* gc, std::list<dcpomatic::Rect<int>> overlaps) override;
 
        int _y; ///< y position in tracks (not pixels)
 };
index 2e6ddc3007c0d9bf886d70f15159ee7138d9214e..7269acebf8d7dff2983e744f8ead8f1affcc7b32 100644 (file)
@@ -30,7 +30,7 @@ class wxSignalManager : public SignalManager
 {
 public:
        explicit wxSignalManager (wxEvtHandler *);
-       void wake_ui ();
+       void wake_ui () override;
 
 private:
        wxEvtHandler* _handler;
index cdd1e2a29871069b7367328c048febfd8580f146..e11423575f48a18b91d70aa020fc0cb1fa6c54c0 100644 (file)
@@ -59,7 +59,7 @@ public:
                set_state (FINISHED_ERROR);
        }
 
-       void run ()
+       void run () override
        {
                while (true) {
                        if (finished ()) {
@@ -68,11 +68,11 @@ public:
                }
        }
 
-       string name () const {
+       string name () const override {
                return "";
        }
 
-       string json_name () const {
+       string json_name () const override {
                return "";
        }
 };