X-Git-Url: https://git.carlh.net/gitweb/?a=blobdiff_plain;ds=sidebyside;f=src%2Fwx%2Fsystem_font_dialog.cc;h=df5b8e21b647a531ec2914592f0079876f4c0cdc;hb=cf7b5fd94e50f740ab93f52343610d01c80ac6d5;hp=ece52e6199834ee21fab85778cf8b72a4ab76473;hpb=3828baf56467224f5d44049bf1e7a7ed11f43a05;p=dcpomatic.git

diff --git a/src/wx/system_font_dialog.cc b/src/wx/system_font_dialog.cc
index ece52e619..df5b8e21b 100644
--- a/src/wx/system_font_dialog.cc
+++ b/src/wx/system_font_dialog.cc
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2015 Carl Hetherington <cth@carlh.net>
+    Copyright (C) 2015-2021 Carl Hetherington <cth@carlh.net>
 
     This file is part of DCP-o-matic.
 
@@ -18,43 +18,40 @@
 
 */
 
+
 #include "system_font_dialog.h"
 #include "wx_util.h"
 #include <wx/listctrl.h>
 #include <boost/filesystem.hpp>
-#include <boost/foreach.hpp>
 #include <iostream>
 
+
 using std::cout;
 using std::string;
 using boost::optional;
 
+
 SystemFontDialog::SystemFontDialog (wxWindow* parent)
 	: wxDialog (parent, wxID_ANY, _("Choose a font"))
 {
-	wxSizer* sizer = new wxBoxSizer (wxVERTICAL);
+	auto sizer = new wxBoxSizer (wxVERTICAL);
 
 	boost::filesystem::path fonts = "c:\\Windows\\Fonts";
-	char* windir = getenv ("windir");
+	auto windir = getenv ("windir");
 	if (windir) {
 		fonts = boost::filesystem::path (windir) / "Fonts";
 	}
 
-	for (
-		boost::filesystem::directory_iterator i = boost::filesystem::directory_iterator (fonts);
-		i != boost::filesystem::directory_iterator ();
-		++i
-		) {
-
-		string ext = i->path().extension().string ();
+	for (auto i: boost::filesystem::directory_iterator (fonts)) {
+		auto ext = i.path().extension().string();
 		transform (ext.begin(), ext.end(), ext.begin(), ::tolower);
 
 		if (ext == ".ttf") {
-			_fonts.push_back (i->path());
+			_fonts.push_back (i.path());
 		}
 	}
 
-	sort (_fonts.begin (), _fonts.end ());
+	sort (_fonts.begin(), _fonts.end());
 
 	_list = new wxListCtrl (this, wxID_ANY, wxDefaultPosition, wxDefaultSize, wxLC_REPORT | wxLC_NO_HEADER);
 	_list->InsertColumn (0, wxT (""));
@@ -62,42 +59,44 @@ SystemFontDialog::SystemFontDialog (wxWindow* parent)
 	sizer->Add (_list, 0, wxALL, DCPOMATIC_SIZER_X_GAP);
 
 	int n = 0;
-	BOOST_FOREACH (boost::filesystem::path i, _fonts) {
+	for (auto i: _fonts) {
 		_list->InsertItem (n++, std_to_wx (i.leaf().stem().string ()));
 	}
 
-	wxSizer* buttons = CreateSeparatedButtonSizer (wxOK | wxCANCEL);
+	auto buttons = CreateSeparatedButtonSizer (wxOK | wxCANCEL);
 	if (buttons) {
 		sizer->Add (buttons, wxSizerFlags().Expand().DoubleBorder());
 	}
 
 	SetSizerAndFit (sizer);
 
-	_list->Bind (wxEVT_COMMAND_LIST_ITEM_SELECTED, boost::bind (&SystemFontDialog::setup_sensitivity, this));
-	_list->Bind (wxEVT_COMMAND_LIST_ITEM_DESELECTED, boost::bind (&SystemFontDialog::setup_sensitivity, this));
+	_list->Bind (wxEVT_LIST_ITEM_SELECTED, boost::bind (&SystemFontDialog::setup_sensitivity, this));
+	_list->Bind (wxEVT_LIST_ITEM_DESELECTED, boost::bind (&SystemFontDialog::setup_sensitivity, this));
 
 	setup_sensitivity ();
 }
 
+
 optional<boost::filesystem::path>
 SystemFontDialog::get_font () const
 {
 	int const s = _list->GetNextItem (-1, wxLIST_NEXT_ALL, wxLIST_STATE_SELECTED);
 	if (s == -1) {
-		return optional<boost::filesystem::path> ();
+		return {};
 	}
 
-	if (s < int (_fonts.size ())) {
+	if (s < int(_fonts.size())) {
 		return _fonts[s];
 	}
 
-	return optional<boost::filesystem::path> ();
+	return {};
 }
 
+
 void
 SystemFontDialog::setup_sensitivity ()
 {
-	wxButton* ok = dynamic_cast<wxButton *> (FindWindowById (wxID_OK, this));
+	auto ok = dynamic_cast<wxButton *> (FindWindowById(wxID_OK, this));
 	if (ok) {
 		ok->Enable (_list->GetNextItem (-1, wxLIST_NEXT_ALL, wxLIST_STATE_SELECTED) != -1);
 	}