X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=src%2Fwx%2Ffilter_dialog.cc;h=3dbb482364bcd6e221e82b88fe99be6c6585f7f5;hb=f984990795e548e2c4160d17d0fa54a40c8b1311;hp=a65f5153a2f95e9733619123997bc2d1180285bc;hpb=a5d004b0773f633401528392fc28e66d70e13ac8;p=dcpomatic.git

diff --git a/src/wx/filter_dialog.cc b/src/wx/filter_dialog.cc
index a65f5153a..3dbb48236 100644
--- a/src/wx/filter_dialog.cc
+++ b/src/wx/filter_dialog.cc
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2012 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2012-2021 Carl Hetherington <cth@carlh.net>
 
     This file is part of DCP-o-matic.
 
@@ -18,10 +18,12 @@
 
 */
 
+
 /** @file  src/filter_dialog.cc
  *  @brief A dialog to select FFmpeg filters.
  */
 
+
 #include "check_box.h"
 #include "filter_dialog.h"
 #include "static_text.h"
@@ -37,35 +39,31 @@ using boost::bind;
 FilterDialog::FilterDialog (wxWindow* parent, vector<Filter const *> const & active)
 	: wxDialog (parent, wxID_ANY, wxString(_("Filters")))
 {
-	wxPanel* panel = new wxPanel (this);
-	wxBoxSizer* sizer = new wxBoxSizer (wxVERTICAL);
+	auto panel = new wxPanel (this);
+	auto sizer = new wxBoxSizer (wxVERTICAL);
 
-	vector<Filter const *> filters = Filter::all ();
+	auto filters = Filter::all ();
 
-	typedef map<string, list<Filter const *> > CategoryMap;
-	CategoryMap categories;
+	map<string, list<Filter const *>> categories;
 
 	for (auto i: filters) {
-		CategoryMap::iterator j = categories.find (i->category());
+		auto j = categories.find (i->category());
 		if (j == categories.end ()) {
-			list<Filter const *> c;
-			c.push_back (i);
-			categories[i->category()] = c;
+			categories[i->category()] = { i };
 		} else {
 			j->second.push_back (i);
 		}
 	}
 
-	for (CategoryMap::iterator i = categories.begin(); i != categories.end(); ++i) {
-
-		wxStaticText* c = new StaticText (panel, std_to_wx(i->first));
-		wxFont font = c->GetFont();
+	for (auto const& i: categories) {
+		auto c = new StaticText (panel, std_to_wx(i.first));
+		auto font = c->GetFont();
 		font.SetWeight(wxFONTWEIGHT_BOLD);
 		c->SetFont(font);
 		sizer->Add (c, 1, wxTOP | wxBOTTOM, DCPOMATIC_SIZER_GAP);
 
-		for (auto j: i->second) {
-			wxCheckBox* b = new CheckBox(panel, std_to_wx(j->name()));
+		for (auto j: i.second) {
+			auto b = new CheckBox(panel, std_to_wx(j->name()));
 			bool const a = find (active.begin(), active.end(), j) != active.end();
 			b->SetValue (a);
 			_filters[j] = b;
@@ -76,14 +74,14 @@ FilterDialog::FilterDialog (wxWindow* parent, vector<Filter const *> const & act
 		sizer->AddSpacer (6);
 	}
 
-	wxSizer* buttons = CreateSeparatedButtonSizer (wxOK);
+	auto buttons = CreateSeparatedButtonSizer (wxOK);
 	if (buttons) {
 		sizer->Add (buttons, wxSizerFlags().Expand().DoubleBorder());
 	}
 
 	panel->SetSizer (sizer);
 
-	wxBoxSizer* overall_sizer = new wxBoxSizer (wxVERTICAL);
+	auto overall_sizer = new wxBoxSizer (wxVERTICAL);
 	overall_sizer->Add (panel, 1, wxTOP | wxLEFT | wxRIGHT, DCPOMATIC_SIZER_GAP);
 	SetSizerAndFit (overall_sizer);
 }
@@ -100,9 +98,9 @@ vector<Filter const*>
 FilterDialog::active () const
 {
 	vector<Filter const *> active;
-	for (map<Filter const *, wxCheckBox*>::const_iterator i = _filters.begin(); i != _filters.end(); ++i) {
-		if (i->second->IsChecked()) {
-			active.push_back(i->first);
+	for (auto const& i: _filters) {
+		if (i.second->IsChecked()) {
+			active.push_back(i.first);
 		}
 	}