Release 0.0.4

Cargo.toml

@@ -2,7 +2,7 @@
 authors = ["Mathieu Rene <[email protected]>"]
 edition = "2018"
 name = "minidsp"
-version = "0.0.3"
+version = "0.0.4"
 license = "MIT"
 description = "A control interface for some MiniDSP products"
 repository = "https://github.com/mrene/minidsp-rs"

README.md

@@ -48,7 +48,7 @@ cargo install minidsp
 
 ## Usage
 
-## Attention
+### Attention
 The changes done through this command will not be visible from the minidsp app, as it cannot read the settings back from the device.
 The following settings will be visible after changing them from any source:
 - Master Gain
@@ -404,4 +404,4 @@ Then reload using:
 
 ```
 sudo udevadm control --reload-rules
-```
+```

src/commands.rs

@@ -414,7 +414,7 @@ pub enum Responses {
     },
 
     /// Speculative commands
-    MaybeConfigChanged,
+    ConfigChanged,
 
     Unknown {
         cmd_id: u8,
@@ -443,7 +443,7 @@ impl Responses {
                     frame.get_u16()
                 },
             },
-            0xab => Responses::MaybeConfigChanged,
+            0xab => Responses::ConfigChanged,
             cmd_id => Responses::Unknown {
                 cmd_id,
                 payload: BytesWrap(frame),
@@ -480,7 +480,7 @@ impl Responses {
                 f.put_u8(0x39);
                 f.put_u16(*size);
             }
-            Responses::MaybeConfigChanged => {}
+            Responses::ConfigChanged => {}
         }
         f.freeze()
     }
@@ -541,6 +541,20 @@ impl Responses {
         }
     }
 
+    pub fn is_config_changed(&self) -> bool {
+        matches!(self, Responses::ConfigChanged)
+    }
+
+    pub fn into_config_changed(self) -> Result<(), MiniDSPError> {
+        match self {
+            Responses::ConfigChanged => Ok(()),
+            _ => Err(MiniDSPError::MalformedResponse(format!(
+                "Expected a ConfigChanged, but got a {:?}",
+                self
+            ))),
+        }
+    }
+
     pub fn is_fir_size(&self) -> bool {
         matches!(self, Responses::FirLoadSize { .. })
     }

src/lib.rs

@@ -165,12 +165,15 @@ impl MiniDSP<'_> {
 
     /// Sets the active configuration
     pub async fn set_config(&self, config: u8) -> Result<()> {
-        self.roundtrip(Commands::SetConfig {
-            config,
-            reset: true,
-        })
-        .await?;
-        Ok(())
+        self.roundtrip_expect(
+            Commands::SetConfig {
+                config,
+                reset: true,
+            },
+            |resp| resp.is_config_changed(),
+        )
+        .await?
+        .into_config_changed()
     }
 
     /// Gets an object wrapping an input channel