[PATCH] Handle seconds in decimal notation (from JDiveLog)

Miika Turkia miika.turkia at gmail.com
Mon Nov 26 20:41:28 PST 2012

On Mon, Nov 26, 2012 at 9:27 PM, Linus Torvalds
<torvalds at linux-foundation.org> wrote:
> On Nov 26, 2012 10:57 AM, "Miika Turkia" <miika.turkia at gmail.com> wrote:
>> In some odd case it is reportedly possible that the
>> seconds are actually 100 based, thus we need to convert that to seconds
>> (1.33333 will become 1 minute 20 seconds).
> I suspect it would be better to also check for "exactly two digits".
> Because if that shit-for-brains jdivelog sometimes uses decimal minutes,
> then six seconds might be done as 0.1 minutes or similar.
> So I think that the check should be a combination of two digits and less
> than sixty...

You are absolutely right. Patch sent.


More information about the subsurface mailing list