C++11 tidying.
authorCarl Hetherington <cth@carlh.net>
Thu, 15 Apr 2021 21:51:08 +0000 (23:51 +0200)
committerCarl Hetherington <cth@carlh.net>
Thu, 15 Apr 2021 23:11:04 +0000 (01:11 +0200)
src/lib/analyse_audio_job.cc
src/lib/dkdm_wrapper.cc
src/lib/dkdm_wrapper.h
src/lib/film.cc
src/lib/hints.cc
src/wx/audio_plot.cc
src/wx/dcp_panel.cc
src/wx/markers_dialog.cc
src/wx/markers_dialog.h

index c94e0b91ff715ebeadc6e9b162cdf29b0800a8a6..448902e1e15f94d6e035cf10757a5a72c075225f 100644 (file)
@@ -205,8 +205,7 @@ AnalyseAudioJob::run ()
                /* If there was only one piece of content in this analysis we may later need to know what its
                   gain was when we analysed it.
                */
-               shared_ptr<const AudioContent> ac = _playlist->content().front()->audio;
-               if (ac) {
+               if (auto ac = _playlist->content().front()->audio) {
                        _analysis->set_analysis_gain (ac->gain());
                }
        }
index 74a71966e87b4667e0ab38b4361556a4ac3b89d9..4698c159d26a97bd9dd85cac1edd153334eacc09 100644 (file)
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2017 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2017-2021 Carl Hetherington <cth@carlh.net>
 
     This file is part of DCP-o-matic.
 
@@ -18,6 +18,7 @@
 
 */
 
+
 #include "compose.hpp"
 #include "dkdm_wrapper.h"
 #include "dcpomatic_assert.h"
@@ -26,78 +27,85 @@ DCPOMATIC_DISABLE_WARNINGS
 #include <libxml++/libxml++.h>
 DCPOMATIC_ENABLE_WARNINGS
 
+
 using std::string;
 using std::list;
 using std::shared_ptr;
+using std::make_shared;
 using std::dynamic_pointer_cast;
 
+
 shared_ptr<DKDMBase>
 DKDMBase::read (cxml::ConstNodePtr node)
 {
        if (node->name() == "DKDM") {
-               return shared_ptr<DKDM> (new DKDM (dcp::EncryptedKDM (node->content ())));
+               return make_shared<DKDM>(dcp::EncryptedKDM(node->content()));
        } else if (node->name() == "DKDMGroup") {
-               shared_ptr<DKDMGroup> group (new DKDMGroup (node->string_attribute ("Name")));
+               auto group = make_shared<DKDMGroup>(node->string_attribute("Name"));
                for (auto i: node->node_children()) {
-                       shared_ptr<DKDMBase> c = read (i);
-                       if (c) {
+                       if (auto c = read(i)) {
                                group->add (c);
                        }
                }
                return group;
        }
 
-       return shared_ptr<DKDMBase> ();
+       return {};
 }
 
+
 string
 DKDM::name () const
 {
        return String::compose ("%1 (%2)", _dkdm.content_title_text(), _dkdm.cpl_id());
 }
 
+
 void
 DKDM::as_xml (xmlpp::Element* node) const
 {
        node->add_child("DKDM")->add_child_text (_dkdm.as_xml ());
 }
 
+
 void
 DKDMGroup::as_xml (xmlpp::Element* node) const
 {
-       xmlpp::Element* f = node->add_child("DKDMGroup");
+       auto f = node->add_child("DKDMGroup");
        f->set_attribute ("Name", _name);
        for (auto i: _children) {
                i->as_xml (f);
        }
 }
 
+
 void
 DKDMGroup::add (shared_ptr<DKDMBase> child, shared_ptr<DKDM> previous)
 {
        DCPOMATIC_ASSERT (child);
        if (previous) {
-               list<shared_ptr<DKDMBase> >::iterator i = find (_children.begin(), _children.end(), previous);
-               if (i != _children.end ()) {
+               auto i = find (_children.begin(), _children.end(), previous);
+               if (i != _children.end()) {
                        ++i;
                }
                _children.insert (i, child);
        } else {
                _children.push_back (child);
        }
-       child->set_parent (dynamic_pointer_cast<DKDMGroup> (shared_from_this ()));
+       child->set_parent (dynamic_pointer_cast<DKDMGroup>(shared_from_this()));
 }
 
+
 void
 DKDMGroup::remove (shared_ptr<DKDMBase> child)
 {
-       for (list<shared_ptr<DKDMBase> >::iterator i = _children.begin(); i != _children.end(); ++i) {
+       for (auto i = _children.begin(); i != _children.end(); ++i) {
                if (*i == child) {
                        _children.erase (i);
-                       child->set_parent (shared_ptr<DKDMGroup> ());
+                       child->set_parent (shared_ptr<DKDMGroup>());
                        return;
                }
-               shared_ptr<DKDMGroup> g = dynamic_pointer_cast<DKDMGroup> (*i);
+               auto g = dynamic_pointer_cast<DKDMGroup> (*i);
                if (g) {
                        g->remove (child);
                }
index 261cdc76751e112b511b49e8b390d8cb21aaa858..0c2992bc41f6f1d413b8eefce6a90b3a1a3a80f6 100644 (file)
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2017 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2017-2021 Carl Hetherington <cth@carlh.net>
 
     This file is part of DCP-o-matic.
 
@@ -18,6 +18,7 @@
 
 */
 
+
 #include <dcp/encrypted_kdm.h>
 #include <libcxml/cxml.h>
 #include <memory>
@@ -27,8 +28,10 @@ namespace xmlpp {
        class Element;
 }
 
+
 class DKDMGroup;
 
+
 class DKDMBase : public std::enable_shared_from_this<DKDMBase>
 {
 public:
@@ -50,6 +53,7 @@ private:
        std::shared_ptr<DKDMGroup> _parent;
 };
 
+
 class DKDM : public DKDMBase
 {
 public:
@@ -68,6 +72,7 @@ private:
        dcp::EncryptedKDM _dkdm;
 };
 
+
 class DKDMGroup : public DKDMBase
 {
 public:
@@ -81,14 +86,14 @@ public:
 
        void as_xml (xmlpp::Element *) const;
 
-       std::list<std::shared_ptr<DKDMBase> > children () const {
+       std::list<std::shared_ptr<DKDMBase>> children () const {
                return _children;
        }
 
-       void add (std::shared_ptr<DKDMBase> child, std::shared_ptr<DKDM> previous = std::shared_ptr<DKDM> ());
+       void add (std::shared_ptr<DKDMBase> child, std::shared_ptr<DKDM> previous = std::shared_ptr<DKDM>());
         void remove (std::shared_ptr<DKDMBase> child);
 
 private:
        std::string _name;
-       std::list<std::shared_ptr<DKDMBase> > _children;
+       std::list<std::shared_ptr<DKDMBase>> _children;
 };
index c57fc165cf9a1b72db957cccd2ac8ef4dff8092f..7f6f8e498eb181d02760131d25e77d25fd80721e 100644 (file)
@@ -284,13 +284,13 @@ Film::audio_analysis_path (shared_ptr<const Playlist> playlist) const
        auto p = dir ("analysis");
 
        Digester digester;
-       for (auto i: playlist->content ()) {
+       for (auto i: playlist->content()) {
                if (!i->audio) {
                        continue;
                }
 
-               digester.add (i->digest ());
-               digester.add (i->audio->mapping().digest ());
+               digester.add (i->digest());
+               digester.add (i->audio->mapping().digest());
                if (playlist->content().size() != 1) {
                        /* Analyses should be considered equal regardless of gain
                           if they were made from just one piece of content.  This
@@ -298,7 +298,7 @@ Film::audio_analysis_path (shared_ptr<const Playlist> playlist) const
                           analysis at the plotting stage rather than having to
                           recompute it.
                        */
-                       digester.add (i->audio->gain ());
+                       digester.add (i->audio->gain());
                }
        }
 
index 729e9a86748be2774eb4c9301d9fd89d928c31af..e9dcfac346c530bac7bcff51fcb6958cbae1b426 100644 (file)
@@ -125,7 +125,7 @@ Hints::check_incorrect_container ()
        int scope = 0;
        for (auto i: film()->content()) {
                if (i->video) {
-                       Ratio const * r = Ratio::nearest_from_ratio(i->video->scaled_size(film()->frame_size()).ratio());
+                       auto const r = Ratio::nearest_from_ratio(i->video->scaled_size(film()->frame_size()).ratio());
                        if (r && r->id() == "239") {
                                ++scope;
                        } else if (r && r->id() != "239" && r->id() != "235" && r->id() != "190") {
index cb10908df821b432dc68692b51c9e1e67fdcc283..f25ae4cb64cb1c8517c13605aa15181971778677 100644 (file)
@@ -18,6 +18,7 @@
 
 */
 
+
 #include "audio_plot.h"
 #include "wx_util.h"
 #include "film_viewer.h"
@@ -29,6 +30,7 @@
 #include <iostream>
 #include <cfloat>
 
+
 using std::cout;
 using std::vector;
 using std::list;
@@ -44,10 +46,12 @@ using namespace boost::placeholders;
 #endif
 using namespace dcpomatic;
 
+
 int const AudioPlot::_minimum = -70;
 int const AudioPlot::_cursor_size = 8;
 int const AudioPlot::max_smoothing = 128;
 
+
 AudioPlot::AudioPlot (wxWindow* parent, weak_ptr<FilmViewer> viewer)
        : wxPanel (parent, wxID_ANY, wxDefaultPosition, wxDefaultSize, wxFULL_REPAINT_ON_RESIZE)
        , _viewer (viewer)
@@ -98,6 +102,7 @@ AudioPlot::AudioPlot (wxWindow* parent, weak_ptr<FilmViewer> viewer)
        SetMinSize (wxSize (640, 512));
 }
 
+
 void
 AudioPlot::set_analysis (shared_ptr<AudioAnalysis> a)
 {
@@ -110,6 +115,7 @@ AudioPlot::set_analysis (shared_ptr<AudioAnalysis> a)
        Refresh ();
 }
 
+
 void
 AudioPlot::set_channel_visible (int c, bool v)
 {
@@ -117,6 +123,7 @@ AudioPlot::set_channel_visible (int c, bool v)
        Refresh ();
 }
 
+
 void
 AudioPlot::set_type_visible (int t, bool v)
 {
@@ -124,6 +131,7 @@ AudioPlot::set_type_visible (int t, bool v)
        Refresh ();
 }
 
+
 void
 AudioPlot::set_message (wxString s)
 {
@@ -131,6 +139,7 @@ AudioPlot::set_message (wxString s)
        Refresh ();
 }
 
+
 struct Metrics
 {
        double db_label_width;
@@ -140,12 +149,13 @@ struct Metrics
        float y_scale;
 };
 
+
 void
 AudioPlot::paint ()
 {
        wxPaintDC dc (this);
 
-       wxGraphicsContext* gc = wxGraphicsContext::Create (dc);
+       auto gc = wxGraphicsContext::Create (dc);
        if (!gc) {
                return;
        }
@@ -157,7 +167,7 @@ AudioPlot::paint ()
                return;
        }
 
-       wxGraphicsPath h_grid = gc->CreatePath ();
+       auto h_grid = gc->CreatePath ();
        gc->SetFont (gc->CreateFont (*wxSMALL_FONT));
        wxDouble db_label_height;
        wxDouble db_label_descent;
@@ -186,7 +196,7 @@ AudioPlot::paint ()
 
        /* Draw an x axis with marks */
 
-       wxGraphicsPath v_grid = gc->CreatePath ();
+       auto v_grid = gc->CreatePath ();
 
        DCPOMATIC_ASSERT (_analysis->samples_per_point() != 0.0);
        double const pps = _analysis->sample_rate() * metrics.x_scale / _analysis->samples_per_point();
@@ -195,7 +205,7 @@ AudioPlot::paint ()
 
        double const mark_interval = calculate_mark_interval (rint (128 / pps));
 
-       DCPTime t = DCPTime::from_seconds (mark_interval);
+       auto t = DCPTime::from_seconds (mark_interval);
        while ((t.seconds() * pps) < data_width) {
                double tc = t.seconds ();
                int const h = tc / 3600;
@@ -204,7 +214,7 @@ AudioPlot::paint ()
                tc -= m * 60;
                int const s = tc;
 
-               wxString str = wxString::Format (wxT ("%02d:%02d:%02d"), h, m, s);
+               auto str = wxString::Format (wxT ("%02d:%02d:%02d"), h, m, s);
                wxDouble str_width;
                wxDouble str_height;
                wxDouble str_descent;
@@ -225,11 +235,11 @@ AudioPlot::paint ()
 
        if (_type_visible[AudioPoint::PEAK]) {
                for (int c = 0; c < MAX_DCP_AUDIO_CHANNELS; ++c) {
-                       wxGraphicsPath p = gc->CreatePath ();
+                       auto p = gc->CreatePath ();
                        if (_channel_visible[c] && c < _analysis->channels()) {
                                plot_peak (p, c, metrics);
                        }
-                       wxColour const col = _colours[c];
+                       auto const col = _colours[c];
                        gc->SetPen (wxPen (wxColour (col.Red(), col.Green(), col.Blue(), col.Alpha() / 2), 1, wxPENSTYLE_SOLID));
                        gc->StrokePath (p);
                }
@@ -237,17 +247,17 @@ AudioPlot::paint ()
 
        if (_type_visible[AudioPoint::RMS]) {
                for (int c = 0; c < MAX_DCP_AUDIO_CHANNELS; ++c) {
-                       wxGraphicsPath p = gc->CreatePath ();
+                       auto p = gc->CreatePath ();
                        if (_channel_visible[c] && c < _analysis->channels()) {
                                plot_rms (p, c, metrics);
                        }
-                       wxColour const col = _colours[c];
+                       auto const col = _colours[c];
                        gc->SetPen (wxPen (col, 1, wxPENSTYLE_SOLID));
                        gc->StrokePath (p);
                }
        }
 
-       wxGraphicsPath axes = gc->CreatePath ();
+       auto axes = gc->CreatePath ();
        axes.MoveToPoint (metrics.db_label_width, 0);
        axes.AddLineToPoint (metrics.db_label_width, metrics.height - metrics.y_origin);
        axes.AddLineToPoint (metrics.db_label_width + data_width, metrics.height - metrics.y_origin);
@@ -255,19 +265,18 @@ AudioPlot::paint ()
        gc->StrokePath (axes);
 
        if (_cursor) {
-               wxGraphicsPath cursor = gc->CreatePath ();
+               auto cursor = gc->CreatePath ();
                cursor.MoveToPoint (_cursor->draw.x - _cursor_size / 2, _cursor->draw.y - _cursor_size / 2);
                cursor.AddLineToPoint (_cursor->draw.x + _cursor_size / 2, _cursor->draw.y + _cursor_size / 2);
                cursor.MoveToPoint (_cursor->draw.x + _cursor_size / 2, _cursor->draw.y - _cursor_size / 2);
                cursor.AddLineToPoint (_cursor->draw.x - _cursor_size / 2, _cursor->draw.y + _cursor_size / 2);
                gc->StrokePath (cursor);
-
-
        }
 
        delete gc;
 }
 
+
 float
 AudioPlot::y_for_linear (float p, Metrics const & metrics) const
 {
@@ -278,6 +287,7 @@ AudioPlot::y_for_linear (float p, Metrics const & metrics) const
        return metrics.height - (linear_to_db(p) - _minimum) * metrics.y_scale - metrics.y_origin;
 }
 
+
 void
 AudioPlot::plot_peak (wxGraphicsPath& path, int channel, Metrics const & metrics) const
 {
@@ -315,6 +325,7 @@ AudioPlot::plot_peak (wxGraphicsPath& path, int channel, Metrics const & metrics
        }
 }
 
+
 void
 AudioPlot::plot_rms (wxGraphicsPath& path, int channel, Metrics const & metrics) const
 {
@@ -384,6 +395,7 @@ AudioPlot::plot_rms (wxGraphicsPath& path, int channel, Metrics const & metrics)
        }
 }
 
+
 void
 AudioPlot::set_smoothing (int s)
 {
@@ -393,6 +405,7 @@ AudioPlot::set_smoothing (int s)
        Refresh ();
 }
 
+
 void
 AudioPlot::set_gain_correction (double gain)
 {
@@ -400,10 +413,11 @@ AudioPlot::set_gain_correction (double gain)
        Refresh ();
 }
 
+
 AudioPoint
 AudioPlot::get_point (int channel, int point) const
 {
-       AudioPoint p = _analysis->get_point (channel, point);
+       auto p = _analysis->get_point (channel, point);
        for (int i = 0; i < AudioPoint::COUNT; ++i) {
                p[i] *= db_to_linear(_gain_correction);
        }
@@ -411,6 +425,7 @@ AudioPlot::get_point (int channel, int point) const
        return p;
 }
 
+
 /** @param n Channel index.
  *  @return Colour used by that channel in the plot.
  */
@@ -421,11 +436,12 @@ AudioPlot::colour (int n) const
        return _colours[n];
 }
 
+
 void
 AudioPlot::search (map<int, PointList> const & search, wxMouseEvent const & ev, double& min_dist, Point& min_point) const
 {
-       for (map<int, PointList>::const_iterator i = search.begin(); i != search.end(); ++i) {
-               for (auto const& j: i->second) {
+       for (auto const& i: search) {
+               for (auto const& j: i.second) {
                        double const dist = pow(ev.GetX() - j.draw.x, 2) + pow(ev.GetY() - j.draw.y, 2);
                        if (dist < min_dist) {
                                min_dist = dist;
@@ -440,8 +456,7 @@ void
 AudioPlot::left_down ()
 {
        if (_cursor) {
-               shared_ptr<FilmViewer> fv = _viewer.lock ();
-               if (fv) {
+               if (auto fv = _viewer.lock()) {
                        fv->seek (_cursor->time, true);
                }
        }
@@ -457,7 +472,7 @@ AudioPlot::mouse_moved (wxMouseEvent& ev)
        search (_rms, ev, min_dist, min_point);
        search (_peak, ev, min_dist, min_point);
 
-       _cursor = optional<Point> ();
+       _cursor = {};
 
        if (min_dist < DBL_MAX) {
                wxRect before (min_point.draw.x - _cursor_size / 2, min_point.draw.y - _cursor_size / 2, _cursor_size, _cursor_size);
@@ -469,10 +484,11 @@ AudioPlot::mouse_moved (wxMouseEvent& ev)
        }
 }
 
+
 void
 AudioPlot::mouse_leave (wxMouseEvent &)
 {
-       _cursor = optional<Point> ();
+       _cursor = {};
        Refresh ();
        Cursor (optional<DCPTime>(), optional<float>());
 }
index 838ccee18e23b6c505f690a6c1692e56d5e7f3d3..3a4644b422d2a14538d4e22c77a2e1614683fb1d 100644 (file)
@@ -895,7 +895,7 @@ DCPPanel::show_audio_clicked ()
 
        if (_audio_dialog) {
                _audio_dialog->Destroy ();
-               _audio_dialog = 0;
+               _audio_dialog = nullptr;
        }
 
        auto d = new AudioDialog (_panel, _film, _viewer);
index 6658fdb4545fb6eb12736a18a04aecf34caa407f..821258a20ec8d874971be3c4a660d2228ddab84e 100644 (file)
@@ -18,6 +18,7 @@
 
 */
 
+
 #include "markers_dialog.h"
 #include "wx_util.h"
 #include "timecode.h"
@@ -31,6 +32,7 @@
 #include <boost/bind/bind.hpp>
 #include <iostream>
 
+
 using std::cout;
 using std::shared_ptr;
 using std::weak_ptr;
@@ -39,6 +41,7 @@ using boost::optional;
 using boost::bind;
 using dcpomatic::DCPTime;
 
+
 class Marker
 {
 public:
@@ -112,6 +115,7 @@ private:
        Button* set_button;
 };
 
+
 MarkersDialog::MarkersDialog (wxWindow* parent, weak_ptr<Film> film, weak_ptr<FilmViewer> viewer)
        : wxDialog (parent, wxID_ANY, _("Markers"))
        , _film (film)
index a3f3532e5b53a9d666f54cb86b06688e0a0b4c43..7d713bd59fbf4f60ff723c1399de7b333b7877c7 100644 (file)
@@ -37,6 +37,6 @@ public:
        MarkersDialog (wxWindow* parent, std::weak_ptr<Film> film, std::weak_ptr<FilmViewer> viewer);
 
 private:
-       std::list<std::shared_ptr<Marker> > _markers;
+       std::list<std::shared_ptr<Marker>> _markers;
        std::weak_ptr<Film> _film;
 };