X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=src%2Fwx%2Ftimer_display.cc;h=a559bb30fe7c8e4f99837a3d1fd3ca1615e80970;hb=31de5b1272d928557a7272977584f5dcc9876c75;hp=7d43a330711e5191ee0c14db4e50d0630346d4a9;hpb=b7e546d9685c0a3304faa48e95516915d811ec5c;p=dcpomatic.git

diff --git a/src/wx/timer_display.cc b/src/wx/timer_display.cc
index 7d43a3307..a559bb30f 100644
--- a/src/wx/timer_display.cc
+++ b/src/wx/timer_display.cc
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2019 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2019-2021 Carl Hetherington <cth@carlh.net>
 
     This file is part of DCP-o-matic.
 
@@ -18,46 +18,44 @@
 
 */
 
+
 #include "timer_display.h"
 #include "wx_util.h"
 #include "lib/timer.h"
 #include <dcp/locale_convert.h>
 #include <list>
 
+
 using std::map;
 using std::list;
 using std::pair;
 using std::make_pair;
 using std::string;
 
-static
-bool
-comparator (pair<string, StateTimer::Counts> const & a, pair<string, StateTimer::Counts> const & b)
-{
-	return a.second.total_time > b.second.total_time;
-}
 
 TimerDisplay::TimerDisplay (wxWindow* parent, StateTimer const & timer, int gets)
 	: TableDialog (parent, std_to_wx(timer.name()), 4, 0, false)
 {
-	map<string, StateTimer::Counts> counts = timer.counts ();
-	list<pair<string, StateTimer::Counts> > sorted;
-	for (map<string, StateTimer::Counts>::const_iterator i = counts.begin(); i != counts.end(); ++i) {
-		sorted.push_back (make_pair(i->first, i->second));
+	auto counts = timer.counts ();
+	list<pair<string, StateTimer::Counts>> sorted;
+	for (auto const& i: counts) {
+		sorted.push_back (make_pair(i.first, i.second));
 	}
 
-	sorted.sort (comparator);
+	sorted.sort ([](pair<string, StateTimer::Counts> const& a, pair<string, StateTimer::Counts> const& b) {
+		return a.second.total_time > b.second.total_time;
+	});
 
 	add (wxString("get() calls"), true);
 	add (std_to_wx(dcp::locale_convert<string>(gets)), false);
 	add_spacer ();
 	add_spacer ();
 
-	for (list<pair<string, StateTimer::Counts> >::const_iterator i = sorted.begin(); i != sorted.end(); ++i) {
-		add (std_to_wx(i->first), true);
-		add (std_to_wx(dcp::locale_convert<string>(i->second.total_time)), false);
-		add (std_to_wx(dcp::locale_convert<string>(i->second.number)), false);
-		add (std_to_wx(dcp::locale_convert<string>(i->second.total_time / i->second.number)), false);
+	for (auto const& i: sorted) {
+		add (std_to_wx(i.first), true);
+		add (std_to_wx(dcp::locale_convert<string>(i.second.total_time)), false);
+		add (std_to_wx(dcp::locale_convert<string>(i.second.number)), false);
+		add (std_to_wx(dcp::locale_convert<string>(i.second.total_time / i.second.number)), false);
 	}
 
 	layout ();