simplify port_mutex_lock_all

liam-teale · ion098 · web-flow · commit e2c2785f053f · 2025-05-20T13:46:34.000-07:00
Co-authored-by: ion098 &lt;146852218+ion098@users.noreply.github.com&gt;
diff --git a/src/system/system_daemon.cpp b/src/system/system_daemon.cpp
@@ -57,9 +57,9 @@ static void unlock_ports(std::index_sequence<Is...>) {
  *
  */
 static void port_mutex_lock_all() {
-    // this looks weird because of how std::lock is implemented. See lock_ports for details.
-    constexpr auto num_ports = std::tuple_size<decltype(zest::Brain::ports)>::value;
-    lock_ports(std::make_index_sequence<num_ports>{});
+    []<size_t... Is>(std::index_sequence<Is...>) {
+        std::lock(zest::Brain::ports[Is].mutex...);
+    }(std::make_index_sequence<zest::Brain::ports.size()>{});
 }
 
 /**

Original file line number	Diff line number	Diff line change
`@@ -57,9 +57,9 @@ static void unlock_ports(std::index_sequence<Is...>) {`
`57`	`57`	`*`
`58`	`58`	`*/`
`59`	`59`	`static void port_mutex_lock_all() {`
`60`		`- // this looks weird because of how std::lock is implemented. See lock_ports for details.`
`61`		`- constexpr auto num_ports = std::tuple_size<decltype(zest::Brain::ports)>::value;`
`62`		`- lock_ports(std::make_index_sequence<num_ports>{});`
	`60`	`+ []<size_t... Is>(std::index_sequence<Is...>) {`
	`61`	`+ std::lock(zest::Brain::ports[Is].mutex...);`
	`62`	`+ }(std::make_index_sequence<zest::Brain::ports.size()>{});`
`63`	`63`	`}`
`64`	`64`
`65`	`65`	`/**`