[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