madhephaestus / ESP32Servo

Arduino-compatible servo library for the ESP32
140 stars 55 forks source link

Disable, or support configurable, Debug output #8

Closed WayneKeenan closed 4 years ago

WayneKeenan commented 5 years ago

Generally it's a good practice that libraries shouldn't output debug, unless requested by user/client code or some kind of other logging settings, as it can interfere with the normal operation of a program.

Please could you remove, or make it a runtime flag to enable/disable (default: disabled), all debug output? e.g. I'm seeing this:

PWM on ledc channel #1 using 'timer 0' to freq 50.00Hz

Thanks

madhephaestus commented 4 years ago

fair point

madhephaestus commented 4 years ago

0.6.2 is released with this change