From a8dbe913b7aa0df0669d6d71283ba656d3183a28 Mon Sep 17 00:00:00 2001
From: Carl Hetherington <cth@carlh.net>
Date: Sun, 21 Mar 2021 18:41:44 +0100
Subject: C++11 tidying.

---
 src/wx/timer_display.cc | 32 +++++++++++++++-----------------
 src/wx/timer_display.h  |  5 ++++-
 2 files changed, 19 insertions(+), 18 deletions(-)

(limited to 'src')

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 ();
diff --git a/src/wx/timer_display.h b/src/wx/timer_display.h
index 5ee72bff5..0fe828c57 100644
--- a/src/wx/timer_display.h
+++ b/src/wx/timer_display.h
@@ -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,10 +18,13 @@
 
 */
 
+
 #include "table_dialog.h"
 
+
 class StateTimer;
 
+
 class TimerDisplay : public TableDialog
 {
 public:
-- 
cgit v1.2.3