<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 10, 2015 at 11:39 PM, 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Tue, Mar 10, 2015 at 11:08:14PM +0200, Yosef Hamza wrote:<br></span><br>
a) the events are ordered by time, so the moment you have one where<br>
   time.seconds > 0 you can stop<br></blockquote><div> </div><div>Yea I thought about that, But Gehad told me before that there was an issue</div><div>that can make events be out of order.</div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
b) even easier, we have a helper function you could use:<br>
        get_next_event(ev, "gaschange");<br>
   so you call this with current_dc->events<br>
   if you get an event back AND its time stamp is 0, remove that event<br>
   try again, again with current_dc->events as pointer<br>
   repeat until you don't find a gas change<br></blockquote><div> </div><div>That's a better solution indeed I didn't know about, I need to surf the helper</div><div>functions more..  </div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
c) I wouldn't call replot() in the middle; while it's unlikely that there<br>
   would be two gas changes at t=0, still one replot() would be sufficient,<br>
   right? So I'd have a little flag (bool eventRemoved) and then call<br>
   mark_divelist_changed() and replot() if eventRemoved is true.<br>
<br>
Makes sense?</blockquote><div><br></div><div>Yes. </div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I know that this might feel frustrating. You keep sending patches and I<br>
keep criticizing the code and not taking them. This really is intended to<br>
help you and to make sure you understand the code and the code we have in<br>
the end makes sense. Sure, I could just fix this myself (that would be a<br>
lot less work for me) - but it's unlikely that you'd learn something if I<br>
did that.<br><span class="HOEnZb"><font color="#888888"><br></font></span></blockquote><div><br></div><div>No not at all, It give me valuable checkmarks when submitting patches :)<br></div><div><br></div><div>§Yousef</div></div><br></div></div>