[PATCH] Only copy used cylinders to planner.

Joakim Bygdell j.bygdell at gmail.com
Sun Feb 15 05:18:44 PST 2015


> On 15 Feb 2015, at 12:31, Robert C. Helling <helling at atdotde.de> wrote:
> 
> Hi,
> 
>> On 15 Feb 2015, at 09:49, Robert C. Helling <helling at atdotde.de <mailto:helling at atdotde.de>> wrote:
>> 
>> thank you for sending this. I had a similar patch some time ago and it was voted down but I would agree that this is the best resolution, so
>> 
>> Ack-by: Robert C. Helling <helling at atdotde.de <mailto:helling at atdotde.de>>
> 
> after some more thought, I think we should let the user decide:
> 
> <0001-Use-unused-cylinders-in-planner-if-they-are-displayed.txt>
> 
> It seems, this is what is intended at other places as well, see void CylindersModel::updateDive()
> 
> 
> Best
> Robert

If we do it this way we also should populate the “Available gases” list with all the gases we import from the parent dive.
Otherwise we are back where we started, that the planner uses gases that are not directly visible to the user.

/Jocke

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.subsurface-divelog.org/pipermail/subsurface/attachments/20150215/b9fa133f/attachment.html>


More information about the subsurface mailing list