Merge pull request #979 from gedankenexperimenter/b/simulator.issue-978

Suppress idle keyswitch events from virtual hardware device

src/kaleidoscope/device/virtual/Virtual.cpp

@@ -260,6 +260,7 @@ void VirtualKeyScanner::actOnMatrixScan() {
       break;
     }
 
+    if (key_state != 0)
       handleKeyswitchEvent(Key_NoKey, key_addr, key_state);
     keystates_prev_[key_addr.toInt()] = keystates_[key_addr.toInt()];
 

testing/bin/ktest-to-cxx

@@ -221,9 +221,7 @@ sub generate_start_new_test {
     cxx("ClearState(); // Clear any state from previous tests");
 }
 sub generate_end_test {
-    if ($reports_expected) {
     generate_check_expected_reports();
-    }
     outdent();
     cxx("} // TEST_F");
     cxx('');

tests/issues/978/common.h

@@ -0,0 +1,27 @@
+// -*- mode: c++ -*-
+
+/* Kaleidoscope - Firmware for computer input devices
+ * Copyright (C) 2020  Keyboard.io, Inc.
+ *
+ * This program is free software: you can redistribute it and/or modify it under
+ * the terms of the GNU General Public License as published by the Free Software
+ * Foundation, version 3.
+ *
+ * This program is distributed in the hope that it will be useful, but WITHOUT
+ * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS
+ * FOR A PARTICULAR PURPOSE. See the GNU General Public License for more
+ * details.
+ *
+ * You should have received a copy of the GNU General Public License along with
+ * this program. If not, see <http://www.gnu.org/licenses/>.
+ */
+
+#pragma once
+
+#include <cstdint>
+
+namespace kaleidoscope {
+namespace testing {
+
+}  // namespace testing
+}  // namespace kaleidoscope

tests/issues/978/sketch.ino

@@ -0,0 +1,72 @@
+/* -*- mode: c++ -*-
+ * Copyright (C) 2020  Keyboard.io, Inc.
+ *
+ * This program is free software: you can redistribute it and/or modify it under
+ * the terms of the GNU General Public License as published by the Free Software
+ * Foundation, version 3.
+ *
+ * This program is distributed in the hope that it will be useful, but WITHOUT
+ * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS
+ * FOR A PARTICULAR PURPOSE. See the GNU General Public License for more
+ * details.
+ *
+ * You should have received a copy of the GNU General Public License along with
+ * this program. If not, see <http://www.gnu.org/licenses/>.
+ */
+
+#include <Kaleidoscope.h>
+
+#include "./common.h"
+
+#undef min
+#undef max
+#include <iostream>
+
+// *INDENT-OFF*
+KEYMAPS(
+    [0] = KEYMAP_STACKED
+    (
+        Key_A, ___, ___, ___, ___, ___, ___,
+        ___, ___, ___, ___, ___, ___, ___,
+        ___, ___, ___, ___, ___, ___,
+        ___, ___, ___, ___, ___, ___, ___,
+        ___, ___, ___, ___,
+        ___,
+
+        ___, ___, ___, ___, ___, ___, ___,
+        ___, ___, ___, ___, ___, ___, ___,
+        ___, ___, ___, ___, ___, ___,
+        ___, ___, ___, ___, ___, ___, ___,
+        ___, ___, ___, ___,
+        ___
+    ),
+)
+// *INDENT-ON*
+
+namespace kaleidoscope {
+namespace plugin {
+
+class IdleKeyDetector : public kaleidoscope::Plugin {
+ public:
+  EventHandlerResult onKeyswitchEvent(Key &key, KeyAddr key_addr, uint8_t key_state) {
+    if (key_addr == KeyAddr{0, 0} && key_state == 0) {
+      handleKeyswitchEvent(Key_X, key_addr, IS_PRESSED | WAS_PRESSED);
+    }
+    return EventHandlerResult::OK;
+  }
+};
+
+} // namespace plugin
+} // namespace kaleidoscope
+
+kaleidoscope::plugin::IdleKeyDetector IdleKeyDetector;
+
+KALEIDOSCOPE_INIT_PLUGINS(IdleKeyDetector);
+
+void setup() {
+  Kaleidoscope.setup();
+}
+
+void loop() {
+  Kaleidoscope.loop();
+}

tests/issues/978/test.ktest

@@ -0,0 +1,12 @@
+VERSION 1
+
+KEYSWITCH A 0 0
+
+# ==============================================================================
+# This testcase shouldn't generate any HID reports. The sketch includes a plugin
+# that produces `Key_X` if the key at (0,0) is idle, so if the virtual device is
+# calling `handleKeyswitchEvent()` when `key_state` has neither `IS_PRESSED` nor
+# `WAS_PRESSED` bits set, it will fail.
+NAME Idle key detector
+
+RUN 3 cycles