djcysmic / NeopixelBusFX

NeopixelBusFX plugin for ESPEasy
27 stars 7 forks source link

_P124_NeoPixelBusFX.ino:957:41: error: 'NeoPixelSendStatus' was not declared in this scope #16

Closed Nobbynews closed 3 years ago

Nobbynews commented 4 years ago

Compiling ESPEasy version mega-20200829 with PlatfromIO i got this error: C:/Daten/ESPEasy/source/src/_P124_NeoPixelBusFX.ino: In function 'boolean Plugin_124(byte, EventStruct*, String&)': C:/Daten/ESPEasy/source/src/_P124_NeoPixelBusFX.ino:862:41: error: 'NeoPixelSendStatus' was not declared in this scope NeoPixelSendStatus(event->Source); ^ C:/Daten/ESPEasy/source/src/_P124_NeoPixelBusFX.ino:957:41: error: 'NeoPixelSendStatus' was not declared in this scope NeoPixelSendStatus(event->Source);

Any idea?

dertobias commented 3 years ago

I got the same error. Commenting these two lines (NeoPixelSendStatus(event->Source)) made build & upload possible and it works without problems. Statusrequest command or any other feedback are now missing as a result.

dertobias commented 3 years ago

Found a cleaner solution: Just cut and paste the JsonResponse part from the end of the code to the top.

`// --------------------------------------------------------------------------------- // ------------------------------ JsonResponse ------------------------------------- // --------------------------------------------------------------------------------- void NeoPixelSendStatus(EventValueSource::Enum eventSource) { String log = String(F("NeoPixelBusFX: Set ")) + rgb.R

compiled with no errors and works like a charm 👍

hope this helps

dertobias commented 3 years ago

All fixed with the latest Version of NeoPixelBusFX and Espeasy. Thanks @clumsy-stefan !

clumsy-stefan commented 3 years ago

n.p.... I'm using the plugin a lot so I'm happy to contribute...

Nobbynews commented 3 years ago

That´s fine, thank you. I will try it soon.