search

thebigpotatoe / Super-Simple-RGB-WiFi-Lamp

A project based on the ESP8266 and WS2812b
MIT License
66 stars 28 forks source link

SaturationFade error #44

Closed StefaanVanDooren closed 4 years ago

StefaanVanDooren commented 4 years ago

One shouldn't be able to set saturationFadeSpeed to 0, because this results in a division by zero an crashes the lamp

int fadeOffset = millis() / (fadeSpeed * 1000 / NUM_LEDS) % NUM_LEDS;

thebigpotatoe commented 4 years ago

Fixed in 851e09f