<div dir="ltr">Hi Dirk and Robert,<br><div><div class="gmail_extra"><br><div class="gmail_quote">On 23 June 2015 at 10:53, Dirk Hohndel <span dir="ltr"><<a href="mailto:dirk@hohndel.org" target="_blank">dirk@hohndel.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
Because of some of the code cleanup that I did over the weekend these were<br>
kinda hard to apply. I tried to close a few dozen Coverity issues and<br>
ended up changing some of those very same conditional statements since<br>
Coverity convinced me that it was possible for nextdp to be NULL...<br>
<br>
Please take a close look at what I just pushed to make sure I didn't end<br>
up breaking things now that you finally got them right.<br>
<br></blockquote><div><br></div><div>With the latest master (and following commit 






<div>
<span style="font-family:monospace"><span style="color:rgb(178,104,24);background-color:rgb(255,255,255)">4094e6b233f967e2bd61f00cef8d53ce23811405</span></span>, which was Robert's second patch, plus the changes to make things build again), we now don't handle the backgas breaks properly when not displaying transitions.  I think the cleanup has left us missing out a gas change where the depth is constant, unless it was entered by the user (i.e. a backgas break).  It did work with my local branch.<br><br></div><div>E.g. 45m depth for 50min bottom time on air, with oxygen as only deco gas (GF = 45/85):<br>depth    duration    runtime    gas<br>45m    3min    3min    air<br>45m    48min    50min     <br>21m    6min    56min     <br>18m    6min    62min     <br>15m    8min    70min     <br>12m    13min    83min     <br>9m    20min    103min     <br>6m    1min    104min    oxygen<br>6m    18min    122min    air    ←-should still be O2 until 116min<br>6m    12min    134min    oxygen<br>6m    6min    140min    air<br>6m    4min    144min    oxygen<br>0m    1min    145min     <br><br></div><div>The profile plot shows that the switch from O2 to air was at 116min.  So does the verbatim output, or table output if transitions are shown.<br></div><div><br></div><div>My local branch got it right:<br>depth    duration    runtime    gas<br>45m    3min    3min    air<br>45m    48min    50min     <br>21m    6min    56min     <br>18m    6min    62min     <br>15m    8min    70min     <br>12m    13min    83min     <br>9m    20min    103min     <br>6m    13min    116min    oxygen<br>6m    6min    122min    air<br>6m    12min    134min    oxygen<br>6m    6min    140min    air<br>6m    4min    144min    oxygen<br>0m    1min    145min     <br></div><div><br></div><div>I *think* the offending change is within the huge stack of conditional statements in lines 644-680.  I believe I have got my head around it and will send a patch tonight my time.<br></div><br></div><div>Rick<br></div></div></div></div></div>