You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
please be aware that at least with some compilers (e.g. MSVC Community 2019 and its bundled toolchain), the C header declaration included as part of the LIFX Protocol V2 Message headers documentation is incorrect in that it doesn't lead to the intended/required packing of the header, specifically the following part of the frame typedef...
I'm unfamiliar with MSVC, what does it complain about? Surely it doesn't matter if it's a uint8 or unit16 for these given they specify only a small portion of bits anyway?
Haven't used MSVC in a while. I think the issue comes in with how the compiler maps the bits, i.e. it is expecting to map all the bits into a uint16_t, i.e. the same uint16_t, and having uint8_t in there as well seems to imply they additional bits should go into another variable which doesn't make sense since there is none. We can definitely look info this.
Hi,
please be aware that at least with some compilers (e.g. MSVC Community 2019 and its bundled toolchain), the C header declaration included as part of the LIFX Protocol V2 Message headers documentation is incorrect in that it doesn't lead to the intended/required packing of the header, specifically the following part of the frame typedef...
...which instead should be all of the same type uint16_t in order not to confuse the compiler...
Thanks for making the LAN protocol option available; keep up the good work!
BR//KHH (@xoblite)
The text was updated successfully, but these errors were encountered: