Missing check for zero height display on Squeezelite-ESP32

[wizmo2]

Squeezelite-ESP32 fails to run when LEDVU is enabled without a display.

The SETD message sends 0 in the display sizes followed by the ledvu strip length. The current code ignores this and still enables display control. The subsequent grfe messages cause a divide-by-zero error in the esp32 code. (other grf messages would also cause issues)

Can we add an additional check to disable render messages if the height (and/or width) is zero?. I can help if needed.

[marcelveldt]

Ah yeah, that part was so badly documented that I just guessed something - I'll make the adjustment, thanks for the tip

[JamiePhonic]

Hi @wizmo2
I submitted a PR (#364) that should fix this issue.
Not sure how long it'll take for it to make it into mainline releases but you can clone this Repo and use the test script to verify. I didn't have any Addressable LED's kicking around to be able to test it myself