[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.

miika


More information about the subsurface mailing list