<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 16 February 2017 at 16:55, 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 Thu, Feb 16, 2017 at 08:14:34AM +1100, Rick Walsh wrote:<br>
> On 16 February 2017 at 07:34, Stefan Fuchs <<a href="mailto:sfuchs@gmx.de">sfuchs@gmx.de</a>> wrote:<br>
><br>
> > Hi All,<br>
> ><br>
> > I stumbled across one small thing:<br>
> ><br>
> > In diveplannermodel.cpp there is a very simple function emitDataChanged()<br>
> > but it is not used consequently. Already a few lines below the same code<br>
> > (which is in fact only one line) is again used instead of calling the<br>
> > function.<br>
> ><br>
> > Is there are reason for this or can we clean this up?<br>
<br>
</span>No reason - thanks for catching that.<br>
I love seeing you look at things like this and taking the initiative.<br>
<span class=""><br>
> emitDataChanged() is also called from desktop-widgets/diveplanner.<wbr>cpp and<br>
> profile-widget/profilewidget2.<wbr>cpp, so I don't think we can get rid of it.<br>
<br>
</span>He did the opposite - he used it more consistently :-)<br></blockquote><div><br></div><div>I thought Stefan's idea was to get rid of the function because it wasn't used.  But on re-reading he email, I realize I missed the point.  Yes, using the function is a good idea.<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
> One of the great things about git is that you can test the consequence of a<br>
> change, so by all means try and see what breaks.  If you work in a new<br>
> branch you can test something, decide if it's a good idea.  If it is you<br>
> make a commit, if not you fix it or checkout master (or your previous<br>
> branch) again and forget about..<br>
<br>
</span>Yep. Git is pretty neat. Using git to maintain Subsurface while running<br>
Linux has a very de-ja-Linus kinda feel to it :-)<br>
<span class="HOEnZb"><font color="#888888"><br>
/D<br>
</font></span></blockquote></div><br></div></div>