From 6191f89d557b9647bea5caaff9666c9437f58f0e Mon Sep 17 00:00:00 2001
From: Robin Gareus <robin@gareus.org>
Date: Sun, 21 Apr 2024 16:31:46 +0200
Subject: [PATCH] Properly update Solo state when setting multiple ctrls

---
 libs/ardour/session_rtevents.cc | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/libs/ardour/session_rtevents.cc b/libs/ardour/session_rtevents.cc
index ca6325d675..e07b231ba0 100644
--- a/libs/ardour/session_rtevents.cc
+++ b/libs/ardour/session_rtevents.cc
@@ -104,13 +104,13 @@ Session::rt_set_controls (std::shared_ptr<WeakAutomationControlList> cl, double
 		return;
 	}
 
-	AutomationType type = NullAutomation;
+	bool update_solo_state = false;
 
 	for (auto const& c : *cl) {
 		std::shared_ptr<AutomationControl> ac = c.lock ();
 		if (ac) {
 			ac->set_value (val, gcd);
-			type = ac->desc().type;
+			update_solo_state |= SoloAutomation == ac->desc().type;
 		}
 	}
 
@@ -118,12 +118,8 @@ Session::rt_set_controls (std::shared_ptr<WeakAutomationControlList> cl, double
 	 * that here.
 	 */
 
-	switch (type) {
-	case SoloAutomation:
+	if (update_solo_state) {
 		update_route_solo_state ();
-		break;
-	default:
-		break;
 	}
 }