On 23 Nov 2014, at 01:22, Dirk Hohndel <[email protected]> wrote:

Dirk,

> Look at the line RIGHT before the first inserted line... I'll move your
> code one line higher :-)

Argh. It’s not good to do these things when tired and a rush.

> Regarding your question if we can simply return from
> fill_missing_tank_pressures() if there is no O2 cylinder... I think we
> should. I actually wonder if we shouldn't have a test way higher up in the
> stack that says something like "if this is a CCR dive and we don't have a
> diluent cylinder and an O2 cylinder, something is seriously messed up...

I think it is fine not having those cylinders defined, like it is fine to have 
an OC dive with no cylinder information at all (in which case we assume to 
breath air and don’t attempt to plot cylinder pressures). 

Similarly, I would assume air as diluent in an unspecified amount and some O2 
cylinder (again without gas usage calculation). I had seriously considered to 
make all arrays of size MAX_CYLINDERS one bigger and reserve 
cylinder[MAX_CYLINDERS] as a secret default O2 cylinder for CCR dives or return 
MAX_CYLINDERS rather than -1 so all code could simply work without error check. 
That option would have been better than calling a random used/defined cylinder 
O2 but I don’t think it would have bought us anything than confusion.

Best
Robert
 
--                                                                              
.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oOo.oO 
Robert C. Helling     Elite Master Course Theoretical and Mathematical Physics  
                      Scientific Coordinator                                   
                      Ludwig Maximilians Universitaet Muenchen, Dept. Physik    
print "Just another   Phone: +49 89 2180-4523  Theresienstr. 39, rm. B339       
    stupid .sig\n";   http://www.atdotde.de 

Attachment: signature.asc
Description: Message signed with OpenPGP using GPGMail

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

Reply via email to