Fixed add negative sign for negative double

[otpfiste]

Hi,

for example that is note rendered correctly:
display.showNumber(-3.1, 1, 3, 1);

The solution is to add the negative sign for negative numbers inside the for loop, see attached code which works for both the 4 and the 6 segments versions:
showNumber-double.txt

All the best,
Oliver

[jasonacox]

Hi @otpfiste ! Great catch! Thank you. I can make this update but do you want to submit a pull request for this change? It would be nice to have you listed as a contributor.

[otpfiste]

Hi jasonacox,

I'm not good at git (still using svn for my projects), it's better if you make the update, I will then test it if you want.

Thanks,
Oliver

[jasonacox]

No problem! Here is the diff (from vscode). I'll update and test for both 4 and 6 digit functions.

Thanks again!

[otpfiste]

Yes, it's correct.

Thanks again for your nice library,
Oliver

[jasonacox]

Thanks, Oliver! I am running test on 4 and 6 digit displays now. I have uploaded the revision to github. I noticed that there was a condition for negative non-floating point numbers that was also not handled so I fixed that as well.

If you can, please test (see 44ac84c). I will upload to the patched version as v1.5.2 for Arduino later today.

[otpfiste]

I updated to v1.5.2 in Arduino IDE and tested it with my 4 and 6 segs displays, all works great!

Note: I re-checked the source code and found several comments to update in TM1637TinyDisplay6.h file:
(0 - leftmost, 3 - rightmost) --> (0 - leftmost, 5 - rightmost)

All the best,
Oliver

[jasonacox]

Thanks, Oliver! I updated the comments as you mentioned.