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
For Color you use an enum which makes code much easier to read as you can just use RED or CYAN to refer to the respective colors. For the sounds, we have to write DuploTrainBaseSound::BRAKE which makes reading a bit more clumsy. It's of course nothing major and maybe I just overlook an important aspect, but if it is exchangeable, why not use a simple enum here as well?
The text was updated successfully, but these errors were encountered:
legoino/src/Lpf2HubConst.h
Line 224 in 434b74f
For
Color
you use an enum which makes code much easier to read as you can just useRED
orCYAN
to refer to the respective colors. For the sounds, we have to writeDuploTrainBaseSound::BRAKE
which makes reading a bit more clumsy. It's of course nothing major and maybe I just overlook an important aspect, but if it is exchangeable, why not use a simple enum here as well?The text was updated successfully, but these errors were encountered: