<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On 16 February 2017 at 07:34, Stefan Fuchs <span dir="ltr"><<a target="_blank" href="mailto:sfuchs@gmx.de">sfuchs@gmx.de</a>></span> wrote:<br><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote">
  

    
  
  <div bgcolor="#FFFFFF">
    <p>Hi All,</p>
    <p>I stumbled across one small thing:</p>
    <p>In diveplannermodel.cpp there is a very simple function
      emitDataChanged() but it is not used consequently. Already a few
      lines below the same code (which is in fact only one line) is
      again used instead of calling the function.</p>
    <p>Is there are reason for this or can we clean this up?<br></p></div></blockquote><div><br></div></div>emitDataChanged() is also called from desktop-widgets/diveplanner.cpp and profile-widget/profilewidget2.cpp, so I don't think we can get rid of it.  One of the great things about git is that you can test the consequence of a change, so by all means try and see what breaks.  If you work in a new branch you can test something, decide if it's a good idea.  If it is you make a commit, if not you fix it or checkout master (or your previous branch) again and forget about..<br><br>I see no reason not to clean up diveplannerlmodel.cpp such that we always use emitDataChanged() instead of emit dataChanged(createIndex(0, 0), createIndex(rowCount() - 1, COLUMNS - 1))<br><br></div><div class="gmail_extra">Cheers,<br><br></div><div class="gmail_extra">Rick<br></div></div>