esp32m / core

ESP32 Manager core
MIT License
26 stars 4 forks source link

Check arg length for response #16

Closed jetpax closed 10 months ago

jetpax commented 10 months ago

Fix response was not correctly generated, and add check for correct arg length

dyarkovoy commented 10 months ago

Can you also move the

uint32_t led_color[3] = { 16, 0, 16 };  // Red, Green, Blue  (0-255) 

from the global scope into the Rmtled class?

Consider the case where we have 2 or more instances of Rmtled on different pins, they will share the same global led_color, which is probably not what we want.

jetpax commented 10 months ago

Indeed, done.