[PATCH] Correct bug: setpoint handling of Poseidon CCR dive logs

Dirk Hohndel dirk at hohndel.org
Wed Oct 29 11:36:13 PDT 2014


On Wed, Oct 29, 2014 at 07:57:38PM +0200, Willem Ferguson wrote:
> @@ -1178,6 +1178,10 @@ static void fixup_dive_dc(struct dive *dive, struct divecomputer *dc)
>  				mintemp = temp;
>  		}
>  
> +		// If there are consecutive identical O2 setpoint readings, throw away the redundant ones.
> +		if (sample->setpoint.mbar == lastsetpoint)
> +				sample->setpoint.mbar = 0;

I believe we came to the conclusion that this was the wrong thing to do
A setpoint of 0 indicates an OC dive...

Can you explain why you need this?

/D


More information about the subsurface mailing list