
<html><head></head><body><div dir="auto">I'm way behind on looking at stuff. I may find time tomorrow, definitely next week on the dive trip.<br><br></div>

<div dir="auto">/D<br><br></div>

<div dir="auto"><!-- tmjah_g_1299s -->-- <!-- tmjah_g_1299e --><br></div>

<div dir="auto"><!-- tmjah_g_1299s -->From my phone<!-- tmjah_g_1299e --></div>

<div  style="font-size:10.0pt;font-family:"Tahoma","sans-serif";padding:3.0pt 0in 0in 0in">

<hr style="border:none;border-top:solid #E1E1E1 1.0pt">

<b>From:</b> "Lubomir I. Ivanov" <neolit123@gmail.com><br>

<b>Sent:</b> Sat Jun 10 13:30:39 PDT 2017<br>

<b>To:</b> Linus Torvalds <torvalds@linux-foundation.org><br>

<b>Cc:</b> Subsurface Mailing List <subsurface@subsurface-divelog.org>, Dirk Hohndel <dirk@hohndel.org><br>

<b>Subject:</b> Re: Github issue 412: lbs values<br>

</dirk@hohndel.org></subsurface@subsurface-divelog.org></torvalds@linux-foundation.org></neolit123@gmail.com></div>

<br>

<pre class="blue">On 10 June 2017 at 23:13, Linus Torvalds <torvalds@linux-foundation.org> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #729fcf; padding-left: 1ex;"> On Sat, Jun 10, 2017 at 12:49 PM, Lubomir I. Ivanov <neolit123@gmail.com> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 1ex 0.8ex; border-left: 1px solid #ad7fa8; padding-left: 1ex;"><br> perhaps both the lrint() calls should be removed and let QString()<br> with 'f' (0/1) handle that?<br></blockquote><br> Agfreed. The whole "lbs = lrint(lbs + 0.05);" thing isn't even a<br> sensible thing to do in the first place, and the other version (that<br> adds half a lbs) is confused too, since lrint() already rounds.<br><br> So ack on your patch.</blockquote><br><br>attached is the signed-off patch with a commit message.<br>maybe Dirk can confirm if we need to handle it differently.<br><br>lubomir<br>--<br></pre></body></html>