[riot-notifications] [RIOT-OS/RIOT] tests/driver_bmp180: cleanup application (#12140)

Alexandre Abadie notifications at github.com
Mon Sep 2 22:10:55 CEST 2019

aabadie commented on this pull request.

>                 "Pressure [hPa]: %lu.%d\n"
                "Pressure at see level [hPa]: %lu.%d\n"
               "Altitude [m]: %i\n"
-               (int)(temperature / 10), (int)(temperature % 10),
+               (int)(temperature / 10), abs(temperature % 10),

In the example provided by @benpicco above, without abs, you would get the following result (tested locally):
Temperature [°C]: -23.-5
But I agree that changing `%d` in `%i` was not necessary...

You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190902/f2f4a520/attachment.htm>

More information about the notifications mailing list