cleanup: remove duplicate null checks.
authorRobin Gareus <robin@gareus.org>
Sun, 28 Feb 2016 11:17:57 +0000 (12:17 +0100)
committerRobin Gareus <robin@gareus.org>
Sun, 28 Feb 2016 11:17:57 +0000 (12:17 +0100)
ARDOUR_UI::save_ardour_state () checks on entry if the windows exist.
and also uses these pointers below without check..

gtk2_ardour/ardour_ui_ed.cc

index 1d9689e608ccca5361296754a4f3449225a3e567..ac8c27faacfce4b97a6ab692c396b2a4757f4210 100644 (file)
@@ -702,11 +702,11 @@ ARDOUR_UI::save_ardour_state ()
 
        string current_tab;
        int current_page_number = _tabs.get_current_page ();
-       if (editor && (current_page_number == _tabs.page_num (editor->contents()))) {
+       if (current_page_number == _tabs.page_num (editor->contents())) {
                current_tab = "editor";
-       } else if (mixer && (current_page_number == _tabs.page_num (mixer->contents()))) {
+       } else if (current_page_number == _tabs.page_num (mixer->contents())) {
                current_tab = "mixer";
-       } else if (rc_option_editor && (current_page_number == _tabs.page_num (rc_option_editor->contents()))) {
+       } else if (current_page_number == _tabs.page_num (rc_option_editor->contents())) {
                current_tab == "preferences";
        }