On Tue, Aug 05, 2014 at 08:48:15PM +0200, Robert C. Helling wrote:
> 
> On 05 Aug 2014, at 16:17, Dirk Hohndel <[email protected]> wrote:
> 
> > void DivePlannerPointsModel::setGFHigh(const int gfhigh)
> > {
> >     tempGFHigh = gfhigh;
> >     // GFHigh <= 34 can cause infinite deco at 6m - don't trigger a
> >     // recalculation
> >     // for smaller GFHigh unless the user explicitly leaves the field
> >     if (tempGFHigh > 34)
> >             triggerGFHigh();
> > }
> 
> Very good. Let’s do this delayed update only if we have 6m as last stop. Here 
> is a patch.

Actually, I think I'd do it differently.
I'd always refuse to do single digit GFHigh until you leave the field.
Because that's what happens when you edit "70"->"7"->"75" and I think
that's 99% of the cases where we get <10.

And if last stop is 6m, refuse anything below 35

Makes sense?

/D
_______________________________________________
subsurface mailing list
[email protected]
http://lists.hohndel.org/cgi-bin/mailman/listinfo/subsurface

Reply via email to