From 01a4a87366493d90b4a85ae354c2502f6572407e Mon Sep 17 00:00:00 2001 From: Carl Hetherington Date: Fri, 17 Jan 2025 01:05:01 +0100 Subject: Check _checked_screens cinema ID instead of checking each screen. --- src/wx/screens_panel.cc | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/src/wx/screens_panel.cc b/src/wx/screens_panel.cc index 3fae01839..630e69494 100644 --- a/src/wx/screens_panel.cc +++ b/src/wx/screens_panel.cc @@ -209,13 +209,11 @@ ScreensPanel::add_cinema(CinemaID cinema_id, Cinema const& cinema, wxTreeListIte auto screens = _cinema_list.screens(cinema_id); if (_show_only_checked->get()) { - auto iter = std::find_if(screens.begin(), screens.end(), [this](pair const& screen) { - auto iter2 = std::find_if(_checked_screens.begin(), _checked_screens.end(), [screen](pair const& checked) { - return checked.second == screen.first; - }); - return iter2 != _checked_screens.end(); + auto iter = std::find_if(_checked_screens.begin(), _checked_screens.end(), [cinema_id](pair const& checked) { + return checked.first == cinema_id; }); - if (iter == screens.end()) { + + if (iter == _checked_screens.end()) { return {}; } } -- cgit v1.2.3