We can (and should) get the screens straight from _checked_screens now.

diff --git a/src/wx/screens_panel.cc b/src/wx/screens_panel.cc
index fbff896ca401581ac47104e75faf5a7438cd3714..69e7f1f4e12df11f76be546c0927c8277e3bb237 100644 (file)
--- a/src/wx/screens_panel.cc
+++ b/src/wx/screens_panel.cc
@@ -401,15 +401,7 @@ vector<shared_ptr<Screen>>
 ScreensPanel::screens () const
 {
        vector<shared_ptr<Screen>> output;
-
-       for (auto item = _targets->GetFirstItem(); item.IsOk(); item = _targets->GetNextItem(item)) {
-               if (_targets->GetCheckedState(item) == wxCHK_CHECKED) {
-                       if (auto screen = item_to_screen(item)) {
-                               output.push_back (screen);
-                       }
-               }
-       }
-
+       std::copy (_checked_screens.begin(), _checked_screens.end(), std::back_inserter(output));
        return output;
 }