Use Focus.readColor in the focusHook

Instead of repeating the same cRGB reading code in three places, use the
recently introduced Focus.readColor() function. This both makes the code
cleaner, and more than 50 bytes smaller too.

Signed-off-by: Gergely Nagy <algernon@keyboard.io>
pull/365/head
Gergely Nagy 6 years ago
parent 73f69c1787
commit 1ab6c016cd

@ -194,9 +194,7 @@ bool LEDControl::focusHook(const char *command) {
} else {
cRGB c;
c.r = Serial.parseInt();
c.g = Serial.parseInt();
c.b = Serial.parseInt();
::Focus.readColor(c);
setCrgbAt(idx, c);
}
@ -205,9 +203,7 @@ bool LEDControl::focusHook(const char *command) {
case SETALL: {
cRGB c;
c.r = Serial.parseInt();
c.g = Serial.parseInt();
c.b = Serial.parseInt();
::Focus.readColor(c);
set_all_leds_to(c);
@ -246,9 +242,7 @@ bool LEDControl::focusHook(const char *command) {
while (idx < LED_COUNT && Serial.peek() != '\n') {
cRGB color;
color.r = Serial.parseInt();
color.g = Serial.parseInt();
color.b = Serial.parseInt();
::Focus.readColor(color);
setCrgbAt(idx, color);
idx++;

Loading…
Cancel
Save