From ab1b72b65c828842123b5baabed0966f549f246d Mon Sep 17 00:00:00 2001 From: Marco Scardovi Date: Mon, 29 Jun 2026 22:17:46 +0200 Subject: [PATCH] fix(asusd): skip failing armoury attributes instead of aborting In start_attributes_zbus, if reload() fails for an attribute, the code logged that it was skipping it, but used break instead of continue. This aborted the entire registration of all subsequent attributes in the directory loop. This patch changes the break to a continue, making individual attribute failures non-fatal, as originally intended. Closes: https://github.com/OpenGamingCollective/asusctl/issues/132 --- asusd/src/asus_armoury.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/asusd/src/asus_armoury.rs b/asusd/src/asus_armoury.rs index d7225b64..e50eeb37 100644 --- a/asusd/src/asus_armoury.rs +++ b/asusd/src/asus_armoury.rs @@ -600,7 +600,7 @@ pub async fn start_attributes_zbus( "Skipping attribute '{}' due to reload error: {e:?}", attr.attr.name() ); - break; + continue; } let attr_name = attr.attribute_name();