<div dir="ltr"><div class="gmail_default" style="font-family:courier new,monospace;font-size:large">Wojciech,<br>Nice, just one minor remark. The commit message should better mention the summary of the bug in the 1st line, i.e.<br>

 "Properly sort newly added tables and queries in forms data source combo"<br>And primarily, it should explain *why* it exists, *not* what is being done. <br><br></div><div class="gmail_default" style="font-family:courier new,monospace;font-size:large">

Always think about commit message as about input to the official changelog.<br><br></div><div class="gmail_default" style="font-family:courier new,monospace;font-size:large"><br>Thiago has explained it nicely:<br><a href="https://plus.google.com/u/0/+ThiagoMacieira/posts/CF5VKSnXXTT">https://plus.google.com/u/0/+ThiagoMacieira/posts/CF5VKSnXXTT</a><br>

<br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 3 June 2014 23:33, Wojciech Kosowicz <span dir="ltr"><<a href="mailto:pcellix@gmail.com" target="_blank">pcellix@gmail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
    
      
    
    <div><div class=""><span><a href="mailto:pcellix@gmail.com" title="Wojciech Kosowicz <pcellix@gmail.com>" target="_blank"> <span>Wojciech Kosowicz</span></a>
</span> changed
              <a title="RESOLVED FIXED - Names of newly added tables and queries not properly sorted in forms data source combo" href="https://bugs.kde.org/show_bug.cgi?id=334093" target="_blank">bug 334093</a>
        <br>
             </div><table border="1" cellpadding="8" cellspacing="0">
          <tbody><tr>
            <th>What</th>
            <th>Removed</th>
            <th>Added</th>
          </tr>

         <tr>
           <td style="text-align:right">Latest Commit</td>
           <td><a href="http://commits.kde.org/calligra/f3949bc88b3a3266980c7e405617dac6a3360c06" target="_blank">http://commits.kde.org/calligra/f3949bc88b3a3266980c7e405617dac6a3360c06</a>
           </td>
           <td><a href="http://commits.kde.org/calligra/5a4e7a9e04273635cd50899f4104c51e37b0a045" target="_blank">http://commits.kde.org/calligra/5a4e7a9e04273635cd50899f4104c51e37b0a045</a>
           </td>
         </tr></tbody></table>
      <p>
        </p><div><div class="">
            <b><a title="RESOLVED FIXED - Names of newly added tables and queries not properly sorted in forms data source combo" href="https://bugs.kde.org/show_bug.cgi?id=334093#c6" target="_blank">Comment # 6</a>
              on <a title="RESOLVED FIXED - Names of newly added tables and queries not properly sorted in forms data source combo" href="https://bugs.kde.org/show_bug.cgi?id=334093" target="_blank">bug 334093</a>
              from <span><a href="mailto:pcellix@gmail.com" title="Wojciech Kosowicz <pcellix@gmail.com>" target="_blank"> <span>Wojciech Kosowicz</span></a>
</span></b>
        </div><pre>Git commit 5a4e7a9e04273635cd50899f4104c51e37b0a045 by Wojciech Kosowicz.
Committed on 03/06/2014 at 21:24.
Pushed by wkosowicz into branch 'master'.

Change of addItem to insertItem in KexiDataSourceComboBox slot

In slotNewItemStored of KexiDataSourceComboBox there was called addItem
method that adds to the end of combobox. Method insertItem should be
called instead so that items are in correct order
FIXED-IN: 2.8.4
REVIEW: 118485

M  +2    -2    kexi/widget/KexiDataSourceComboBox.cpp

<a href="http://commits.kde.org/calligra/5a4e7a9e04273635cd50899f4104c51e37b0a045" target="_blank">http://commits.kde.org/calligra/5a4e7a9e04273635cd50899f4104c51e37b0a045</a></pre>
        </div>
      <p></p><div class="">
      <hr>
      <span>You are receiving this mail because:</span>
      
      <ul>
          <li>You are on the CC list for the bug.</li>
          <li>You are watching someone on the CC list of the bug.</li>
          <li>You are watching the assignee of the bug.</li>
      </ul>
    </div></div>

</blockquote></div><br><br clear="all"><br>-- <br>regards / pozdrawiam, Jaroslaw Staniek<br> Kexi & Calligra & KDE | <a href="http://calligra.org/kexi" target="_blank">http://calligra.org/kexi</a> | <a href="http://kde.org" target="_blank">http://kde.org</a><br>

 Qt for Tizen | <a href="http://qt-project.org/wiki/Tizen" target="_blank">http://qt-project.org/wiki/Tizen</a><br> Qt Certified Specialist | <a href="http://www.linkedin.com/in/jstaniek" target="_blank">http://www.linkedin.com/in/jstaniek</a><br>


</div>