Small temperature import bug

Henrik Brautaset Aronsen subsurface at henrik.synth.no
Sun Feb 17 12:13:26 PST 2013


Linus Torvalds wrote:
> Anyway, if I'm right, something like the attached (BUT TOTALLY
> UNTESTED) patch that simply moves the fixup_dive_dc() code up migth
> fix it.
>
> But maybe the fixup_dive_dc() code already relies on some of the
> values having been fixed up. That's why I say "totally untested",
> there might be some other ordering dependency that I didn't notice.
>
> Dirk? Not-signed-off-yet-due-to-lack-of-testing, but if Henrik acks
> it, feel free to add my sign-off.

I can't guarantee any unwanted side-effects, but it looks good to go.  
Temperatures show up immediately after the import, and the resulting 
saved XML is identical to the XML before patching.

Acked-by: Henrik Brautaset Aronsen <subsurface at henrik.synth.no>

H
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.hohndel.org/pipermail/subsurface/attachments/20130217/98e967b8/attachment.html>


More information about the subsurface mailing list