Duplicate events fix (#17)

* Filter duplicate events Signed-off-by: Sami Salonen <[email protected]> * Cleanup Signed-off-by: Sami Salonen <[email protected]> * Filter zero duration events only with Mute, not Vol up/down Signed-off-by: Sami Salonen <[email protected]> --------- Signed-off-by: Sami Salonen <[email protected]>
ssalonen · Mar 26, 2023 · 67a914d · 67a914d
1 parent 7bfb9ef
commit 67a914d
Show file tree

Hide file tree

Showing 4 changed files with 19 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -7,6 +7,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## 3.1.1
+
+- Filter duplicate key press events (zero duration)
+
 ## 3.1.0
 
 

diff --git a/Cargo.lock b/Cargo.lock
diff --git a/Cargo.toml b/Cargo.toml
@@ -4,7 +4,7 @@ path = 'src/bin.rs'
 
 [package]
 name = 'cec-alsa-sync'
-version = "3.1.0"
+version = "3.1.1"
 authors = ['Sami Salonen']
 edition = '2018'
 license = 'GPL-2.0'

diff --git a/src/bin.rs b/src/bin.rs
@@ -77,10 +77,22 @@ fn on_key_press(keypress: CecKeypress) {
     let command: Option<Command> = match keypress.keycode {
         CecUserControlCode::VolumeUp => Some(app_config.vol_up_command.new_command()),
         CecUserControlCode::VolumeDown => Some(app_config.vol_down_command.new_command()),
-        CecUserControlCode::Mute => app_config.mute_command.as_ref().map(|c| c.new_command()),
+        CecUserControlCode::Mute => {
+            if keypress.duration.is_zero() {
+                // Filter duplicate events
+                None
+            } else {
+                app_config.mute_command.as_ref().map(|c| c.new_command())
+            }
+        }
         _ => None,
     };
     if let Some(mut command) = command {
+        debug!(
+            "Executing command {:?} {:?}",
+            command.get_program(),
+            command.get_args()
+        );
         command.output().expect("Failed to call amixer");
     }
 }