test binaries for Beta 5

Dirk Hohndel dirk at hohndel.org
Tue Aug 5 12:53:12 PDT 2014


On Tue, Aug 05, 2014 at 08:48:15PM +0200, Robert C. Helling wrote:
> 
> On 05 Aug 2014, at 16:17, Dirk Hohndel <dirk at hohndel.org> 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


More information about the subsurface mailing list