diff options
author | XScorpion2 <rcalt2vt@gmail.com> | 2019-05-01 13:36:50 -0500 |
---|---|---|
committer | MechMerlin <30334081+mechmerlin@users.noreply.github.com> | 2019-05-01 11:36:50 -0700 |
commit | 3235c8527dd292d4a5e5e28302e118f3ef1a043e (patch) | |
tree | dd1c013612ed0f4379653b0e7171d439bf140827 | |
parent | 22ba36a4d86c06ce1b6049555525646eb16bb57a (diff) | |
download | qmk_firmware-3235c8527dd292d4a5e5e28302e118f3ef1a043e.tar.gz qmk_firmware-3235c8527dd292d4a5e5e28302e118f3ef1a043e.zip |
Simple fix for selecting which tracked key press to work off of for simple reactive effects (#5745)
-rw-r--r-- | quantum/rgb_matrix_animations/solid_reactive_anim.h | 3 | ||||
-rw-r--r-- | quantum/rgb_matrix_animations/solid_reactive_simple_anim.h | 3 |
2 files changed, 4 insertions, 2 deletions
diff --git a/quantum/rgb_matrix_animations/solid_reactive_anim.h b/quantum/rgb_matrix_animations/solid_reactive_anim.h index 836409dd6..82483653a 100644 --- a/quantum/rgb_matrix_animations/solid_reactive_anim.h +++ b/quantum/rgb_matrix_animations/solid_reactive_anim.h | |||
@@ -15,7 +15,8 @@ bool rgb_matrix_solid_reactive(effect_params_t* params) { | |||
15 | for (uint8_t i = led_min; i < led_max; i++) { | 15 | for (uint8_t i = led_min; i < led_max; i++) { |
16 | RGB_MATRIX_TEST_LED_FLAGS(); | 16 | RGB_MATRIX_TEST_LED_FLAGS(); |
17 | uint16_t tick = max_tick; | 17 | uint16_t tick = max_tick; |
18 | for(uint8_t j = 0; j < g_last_hit_tracker.count; j++) { | 18 | // Reverse search to find most recent key hit |
19 | for (int8_t j = g_last_hit_tracker.count - 1; j >= 0; j--) { | ||
19 | if (g_last_hit_tracker.index[j] == i && g_last_hit_tracker.tick[j] < tick) { | 20 | if (g_last_hit_tracker.index[j] == i && g_last_hit_tracker.tick[j] < tick) { |
20 | tick = g_last_hit_tracker.tick[j]; | 21 | tick = g_last_hit_tracker.tick[j]; |
21 | break; | 22 | break; |
diff --git a/quantum/rgb_matrix_animations/solid_reactive_simple_anim.h b/quantum/rgb_matrix_animations/solid_reactive_simple_anim.h index d7bdb414e..9fc4d527a 100644 --- a/quantum/rgb_matrix_animations/solid_reactive_simple_anim.h +++ b/quantum/rgb_matrix_animations/solid_reactive_simple_anim.h | |||
@@ -14,7 +14,8 @@ bool rgb_matrix_solid_reactive_simple(effect_params_t* params) { | |||
14 | for (uint8_t i = led_min; i < led_max; i++) { | 14 | for (uint8_t i = led_min; i < led_max; i++) { |
15 | RGB_MATRIX_TEST_LED_FLAGS(); | 15 | RGB_MATRIX_TEST_LED_FLAGS(); |
16 | uint16_t tick = max_tick; | 16 | uint16_t tick = max_tick; |
17 | for(uint8_t j = 0; j < g_last_hit_tracker.count; j++) { | 17 | // Reverse search to find most recent key hit |
18 | for (int8_t j = g_last_hit_tracker.count - 1; j >= 0; j--) { | ||
18 | if (g_last_hit_tracker.index[j] == i && g_last_hit_tracker.tick[j] < tick) { | 19 | if (g_last_hit_tracker.index[j] == i && g_last_hit_tracker.tick[j] < tick) { |
19 | tick = g_last_hit_tracker.tick[j]; | 20 | tick = g_last_hit_tracker.tick[j]; |
20 | break; | 21 | break; |