Simple fix for selecting which tracked key press to work off of for simple reactive effects (#5745)
This commit is contained in:
parent
22ba36a4d8
commit
3235c8527d
2 changed files with 4 additions and 2 deletions
|
@ -15,7 +15,8 @@ bool rgb_matrix_solid_reactive(effect_params_t* params) {
|
||||||
for (uint8_t i = led_min; i < led_max; i++) {
|
for (uint8_t i = led_min; i < led_max; i++) {
|
||||||
RGB_MATRIX_TEST_LED_FLAGS();
|
RGB_MATRIX_TEST_LED_FLAGS();
|
||||||
uint16_t tick = max_tick;
|
uint16_t tick = max_tick;
|
||||||
for(uint8_t j = 0; j < g_last_hit_tracker.count; j++) {
|
// Reverse search to find most recent key hit
|
||||||
|
for (int8_t j = g_last_hit_tracker.count - 1; j >= 0; j--) {
|
||||||
if (g_last_hit_tracker.index[j] == i && g_last_hit_tracker.tick[j] < tick) {
|
if (g_last_hit_tracker.index[j] == i && g_last_hit_tracker.tick[j] < tick) {
|
||||||
tick = g_last_hit_tracker.tick[j];
|
tick = g_last_hit_tracker.tick[j];
|
||||||
break;
|
break;
|
||||||
|
|
|
@ -14,7 +14,8 @@ bool rgb_matrix_solid_reactive_simple(effect_params_t* params) {
|
||||||
for (uint8_t i = led_min; i < led_max; i++) {
|
for (uint8_t i = led_min; i < led_max; i++) {
|
||||||
RGB_MATRIX_TEST_LED_FLAGS();
|
RGB_MATRIX_TEST_LED_FLAGS();
|
||||||
uint16_t tick = max_tick;
|
uint16_t tick = max_tick;
|
||||||
for(uint8_t j = 0; j < g_last_hit_tracker.count; j++) {
|
// Reverse search to find most recent key hit
|
||||||
|
for (int8_t j = g_last_hit_tracker.count - 1; j >= 0; j--) {
|
||||||
if (g_last_hit_tracker.index[j] == i && g_last_hit_tracker.tick[j] < tick) {
|
if (g_last_hit_tracker.index[j] == i && g_last_hit_tracker.tick[j] < tick) {
|
||||||
tick = g_last_hit_tracker.tick[j];
|
tick = g_last_hit_tracker.tick[j];
|
||||||
break;
|
break;
|
||||||
|
|
Loading…
Reference in a new issue