<p dir="ltr">Ups. I will fix those right away -  and yes I tested closing and reopening the dive site a bunch of times, don't know what can be the issue. Let me take a look </p>
<br><div class="gmail_quote">On Sun, May 17, 2015, 18:09 Dirk Hohndel <<a href="mailto:dirk@hohndel.org">dirk@hohndel.org</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Sun, May 17, 2015 at 05:16:04PM -0300, Tomaz Canabrava wrote:<br>
> plus a few more. :)<br>
> I ll also send a bunch more patches today<br>
> but now, lunch and movie. :D<br>
<br>
Enjoy the movie. And thanks for being back working on Subsurface.<br>
<br>
Quick question - did you build test these patches? There was a missing<br>
#include and a stray letter 'a' - both of which made the build fail quite<br>
obviously and reproducably...<br>
<br>
And once I fix those, I can rather easily reproduce a crash:<br>
Open dive file. Click Location-manage. Pick a site. Click close. Click<br>
Location-manage again. BOOM. I added a patch that seems to make sense, but<br>
since not a lot of the code is hooked up yet, I'm not sure this is what we<br>
want in the end.<br>
<br>
I was debating whether to add this to master, but I'd rather have you<br>
continue on top of this than on top what you had sent, so everyone else,<br>
be careful, right now master has some issues :-/<br>
<br>
/D<br>
</blockquote></div>