On Wed, Oct 29, 2014 at 10:10:56PM +0200, Willem Ferguson wrote: > > > >I think in memory we should NOT set anything to 0. When the setpoint > >switches to 0 that means we are bailing out to OC... > > In fixup_dive_dc() there has always been some zeroing out from the > beginning, I m reasonably sure in the original code from Linus. But that > related to tank pressures and temperature, not oxygen values. In that > function I added some zeroing out of oxygen-related variables in an attempt > to be consistent. Your argument makes good sense. Should I only remove > zeroing out for the setpoint or should I do for all oxygen-related > variables?
There should be no zeroing out. fixup_dive_dc() clears tank pressures if we switched away from that tank. That's the only thing it should do. If it does more than that, please point it out to me. /D _______________________________________________ subsurface mailing list subsurface@subsurface-divelog.org http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface