]> git.donarmstrong.com Git - qmk_firmware.git/commitdiff
dynamic_macro.h: Fix an off-by-two error
authorWojciech Siewierski <wojciech.siewierski@onet.pl>
Wed, 3 May 2017 23:19:05 +0000 (01:19 +0200)
committerWojciech Siewierski <wojciech.siewierski@onet.pl>
Wed, 3 May 2017 23:26:35 +0000 (01:26 +0200)
We need to check whether we just passed the after-the-end point of the
other macro. Instead we were checking whether we are going to reach it
now.

quantum/dynamic_macro.h

index c9120897f8fecf030e5db2d5ebe6e5de102c0e28..9e7845c9928563be754a16101e0f353c632a5d9d 100644 (file)
@@ -99,14 +99,14 @@ void dynamic_macro_play(
  *
  * @param macro_buffer[in] The start of the used macro buffer.
  * @param macro_pointer[in,out] The current buffer position.
- * @param macro_end2[in] The end of the other macro which shouldn't be overwritten.
+ * @param macro2_end[in] The last buffer element it is safe to use before overwriting the other macro.
  * @param direction[in]  Either +1 or -1, which way to iterate the buffer.
  * @param record[in]     The current keypress.
  */
 void dynamic_macro_record_key(
     keyrecord_t *macro_buffer,
     keyrecord_t **macro_pointer,
-    keyrecord_t *macro_end2,
+    keyrecord_t *macro2_end,
     int8_t direction,
     keyrecord_t *record)
 {
@@ -115,7 +115,7 @@ void dynamic_macro_record_key(
         return;
     }
 
-    if (*macro_pointer + direction != macro_end2) {
+    if (*macro_pointer - direction != macro2_end) {
         **macro_pointer = *record;
         *macro_pointer += direction;
     } else {