Set_self_solo should always set the mute_master state as well.

This fixed a bug in mixbus when a session was reopened with a soloed bus.
This may not be necessary in Ardour but I think it is more correct.

diff --git a/libs/ardour/route.cc b/libs/ardour/route.cc
index 48661794903985b953f9bc3d114cd47cdbe12978..53630d4e9ccb83cdc3fa2589de4fb5ebe11738cb 100644 (file)
--- a/libs/ardour/route.cc
+++ b/libs/ardour/route.cc
@@ -890,7 +890,6 @@ Route::set_solo (bool yn, void *src, bool group_override)
 
        if (self_soloed() != yn) {
                set_self_solo (yn);
-               set_mute_master_solo ();
                solo_changed (true, src, group_override); /* EMIT SIGNAL */
                _solo_control->Changed (); /* EMIT SIGNAL */
        }
@@ -911,6 +910,7 @@ Route::set_self_solo (bool yn)
 {
        DEBUG_TRACE (DEBUG::Solo, string_compose ("%1: set SELF solo => %2\n", name(), yn));
        _self_solo = yn;
+       set_mute_master_solo ();
 }
 
 void