<table><tr><td style="">dfaure accepted this revision.<br />dfaure added a comment.<br />This revision is now accepted and ready to land.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D26440">View Revision</a></tr></table><br /><div><div><p>Looks good. I'm just going back and forth in my mind between dolphinstate and dolphinstaterc. The latter would have the benefit of telling experienced developers that it uses the KConfig file format (not e.g. some binary format). This is minor though, surely we'll get used to it whichever way.</p>

<p>Maybe mention the method name in the first line of the commit log, so that the future release changelog is more informative.</p>

<p>Thanks!</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R237 KConfig</div></div></div><br /><div><strong>BRANCH</strong><div><div>state</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D26440">https://phabricator.kde.org/D26440</a></div></div><br /><div><strong>To: </strong>nicolasfella, Frameworks, dfaure<br /><strong>Cc: </strong>kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns<br /></div>