[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"
                "\n+-------------------------------------+\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:
https://github.com/RIOT-OS/RIOT/pull/12140#discussion_r320040963
-------------- 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