[PATCH] Clean up Divesoft Freedome time parsing
Dirk Hohndel
dirk at hohndel.org
Sat Oct 3 07:41:44 PDT 2015
On Sat, Oct 03, 2015 at 10:36:11AM -0400, Linus Torvalds wrote:
>
> This is a bigger patch than required (it could have just changed the
> "uint32_t" into "int32_t"), but I think the code ends up being easier to
> read, and it should be as efficient (although because we compile with
> debugging and no optimizations, splitting things up like this ends up
> making the compiler split things up too - but that's our debug thing,
> not anything fundamental).
I'm obviously not one bit worried about the overhead, but I actually like
this much better as it is clean and obvious.
Thanks, Linus
/D
More information about the subsurface
mailing list