jgarff / rpi_ws281x

Userspace Raspberry Pi PWM library for WS281X LEDs
BSD 2-Clause "Simplified" License
1.76k stars 615 forks source link

Correcting designator order in main.c #497

Closed Deadolus closed 2 years ago

Deadolus commented 2 years ago

The designator order in main.c does not match the one defined in ws2811.h. This leads to problems when using the code with c++ compilers.

I corrected the initialization order here and the compilation now works with the afromentioned complainers without issues. Tested on Ubuntu 21.10 (clang 13.0.0-2/gcc 11.2.0) and Raspberry Pi 4 Release 11 bullseye (GCC 10.2.1)

Gadgetoid commented 2 years ago

Thank you. This still gets me basically every time in C++.