diff --git a/CLAUDE.md b/CLAUDE.md index b6be638..e2c5422 100644 --- a/CLAUDE.md +++ b/CLAUDE.md @@ -90,7 +90,7 @@ Commands arrive as newline-terminated ASCII lines. Each `parse*` function in `pr | `HOME` | `HOME ` / `HOMEALL` | Run homing sequence | | `SWEEP` | `SWEEP ` | Start sweep (0/0 stops) | | `POS?` | `POS?` | Query all gauges: `POS ` | -| `LED` | `LED ` | Set one LED (0-based index within gauge segment) to RGB colour (0–255 each) | +| `LED` | `LED ` | Set one LED (0-based index within gauge segment) to RGB colour (0–255 each); `` may be a range `N-M` to set LEDs N through M in one command | | `LED?` | `LED?` | Query all LEDs: one `LED ` line per LED, then `OK` | | `PING` | `PING` | Responds `PONG` | diff --git a/Gaugecontroller/Gaugecontroller.ino b/Gaugecontroller/Gaugecontroller.ino index 55b6407..139c3aa 100644 --- a/Gaugecontroller/Gaugecontroller.ino +++ b/Gaugecontroller/Gaugecontroller.ino @@ -520,13 +520,19 @@ bool parseLedQuery(const String& line) { } bool parseLed(const String& line) { - int id, idx, r, g, b; - if (sscanf(line.c_str(), "LED %d %d %d %d %d", &id, &idx, &r, &g, &b) == 5) { + int id, r, g, b; + char idxToken[16]; + if (sscanf(line.c_str(), "LED %d %15s %d %d %d", &id, idxToken, &r, &g, &b) == 5) { if (id < 0 || id >= GAUGE_COUNT) { sendReply("ERR BAD_ID"); return true; } - if (idx < 0 || idx >= gaugePins[id].ledCount) { sendReply("ERR BAD_IDX"); return true; } - leds[gaugeLedOffset[id] + idx] = CRGB(constrain(r, 0, 255), - constrain(g, 0, 255), - constrain(b, 0, 255)); + char* dash = strchr(idxToken, '-'); + int idxFirst = atoi(idxToken); + int idxLast = dash ? atoi(dash + 1) : idxFirst; + if (idxFirst < 0 || idxLast >= gaugePins[id].ledCount || idxFirst > idxLast) { + sendReply("ERR BAD_IDX"); return true; + } + CRGB color(constrain(r, 0, 255), constrain(g, 0, 255), constrain(b, 0, 255)); + for (int i = idxFirst; i <= idxLast; i++) + leds[gaugeLedOffset[id] + i] = color; FastLED.show(); sendReply("OK"); return true;