Hi Robert,

On Donnerstag, 2. Dezember 2021 08:38:25 CET Robert Helling via subsurface 
wrote:

>               fn2 = (int)(1000.0 * entry->pressures.n2 / 
amb_pressure);

This looks ill-defined. You probably want lrint(...) or (int)(round(...)) or  
(int)(ceil(...)) or (int)(floor(...)) for reproducible behavior.

The whole code looks fishy to me (why round to int in the first place?).

Berthold


_______________________________________________
subsurface mailing list
subsurface@subsurface-divelog.org
http://lists.subsurface-divelog.org/cgi-bin/mailman/listinfo/subsurface

Reply via email to