BojanJurca / Esp32_oscilloscope

ESP32 oscilloscope - see the signals through Web browser the way ESP32 sees them
Creative Commons Zero v1.0 Universal
741 stars 84 forks source link

[oscilloscope] Use enums instead of strcmp #23

Open kubark42 opened 10 months ago

kubark42 commented 10 months ago

UNTESTED: DO NOT MERGE


Using enums is faster, smaller, and safer than string comparisons. This also puts the code validators as close to the parsers as possible.


I'm floating a trial balloon to see if this kind of modification would be appreciated. If so, I can do some testing to prep this for merge.