Fix esp8266 crash, int overflow and compiler warnings

ph1p / ikea-led-obegraensad

ESP32/Arduino hack for the ikea OBEGRÄNSAD led wall lamp

MIT License

578 stars 78 forks source link

Closed jekkos closed 1 year ago

jekkos commented 1 year ago

@ph1p This PR fixes a couple of issues

Linking of the executable failed in case you disable the storage option
Compiler warnings in the snake animation because of mixed usage of int and unsigned ints
Removed a workaround for what was in fact an integer overflow issue
Fix for a crash on esp8266, due to a delay() statement when switching modes. Any idea why it was there?

ph1p commented 1 year ago

Thank you! :) I added int in the screen.cpp again, because it didn`t work on my ESP32 :/ Does it work with this change on your ESP8266?