NanoDMX/StageProfi/Enttec fix

[brianwiddas]

For any value over 127, the nanodmx code sends the update to the device repeatedly rather than just once, as happens for any value <= 127

This is caused by a signed/unsigned value comparison in nanodmx.cpp

Fixed and tested. Also fixed the same code in stageprofi.cpp and enttecdmxusbpro.cpp, but not tested.

[mcallegari]

Thanks for the patch, makes sense!
However, looking at the code, maybe the right way is to declare 'val' as char, since it comes and goes into QByteArrays.
What do you think?

[coveralls]

coverage: 28.054%. remained the same when pulling 023552a on brianwiddas:nanodmx-compare into 04956e8 on mcallegari:master.

[brianwiddas]

Hi,

Yes, that makes sense. I looked over the code for all three and can't see any reason why it would be a problem.

Updated the code and tested nanodmx

[mcallegari]

Thanks 👍