First step in cleaning up cylinder pressure sensor logic

Dirk Hohndel dirk at
Tue Jan 1 08:33:48 PST 2013

Linus Torvalds <torvalds at> writes:

> On Sun, Dec 30, 2012 at 9:46 PM, Linus Torvalds
> <torvalds at> wrote:
>> Looks good, along with your nickname quoting fix.
> Oops. I take that back. The nickname quoting fix is buggered.
> The reason I thought it was fine is that I don't get invalid XML any
> more - good. But that turns out to be because the nickname didn't get
> saved at all - bad.
> I don't think you call "clear_dc_saved()" anywhere, so nn_entry->saved
> has some random stale value that can be true, so now it never gets
> saved at all.
> Adding a call to clear_dc_saved() at the top of save_dives() fixes it for me.

Yes. It worked through half a dozen tests - pure luck.

Fixed and pushed. Thanks for noticing right away!


More information about the subsurface mailing list