<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/113045/">http://git.reviewboard.kde.org/r/113045/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On October 4th, 2013, 7 a.m. UTC, <b>David Faure</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Looks good, but why don't we replace KCategoryDrawer with KCategoryDrawerV3, in itemviews? I.e. getting rid of the two oldest ones?
It should be SC since v3 derives from v2 derives from v1.</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">It's not entirely source compatible since the constructor changes and v1 needs deleting manually.
Will be easy to change now that everything in kdelibs uses V3 style. Will make a new review today.</pre>
<br />
<p>- David</p>
<br />
<p>On October 2nd, 2013, 8:06 a.m. UTC, David Edmundson wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for KDE Frameworks.</div>
<div>By David Edmundson.</div>
<p style="color: grey;"><i>Updated Oct. 2, 2013, 8:06 a.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
kdelibs
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Only find publicly used libraries in cmake.in
KCategoryDrawer -> KCategoryDrawerV3
No QString cast from ascii
</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>staging/kcmutils/CMakeLists.txt <span style="color: grey">(dc577d3)</span></li>
<li>staging/kcmutils/KCMUtilsConfig.cmake.in <span style="color: grey">(c67ad52)</span></li>
<li>staging/kcmutils/src/CMakeLists.txt <span style="color: grey">(dc19649)</span></li>
<li>staging/kcmutils/src/kcmodulecontainer.cpp <span style="color: grey">(c08eda2)</span></li>
<li>staging/kcmutils/src/kcmoduleinfo.cpp <span style="color: grey">(f46308c)</span></li>
<li>staging/kcmutils/src/kcmoduleproxy.cpp <span style="color: grey">(d2db3ab)</span></li>
<li>staging/kcmutils/src/kcmultidialog.cpp <span style="color: grey">(4812407)</span></li>
<li>staging/kcmutils/src/kpluginselector.cpp <span style="color: grey">(77cbfda)</span></li>
<li>staging/kcmutils/src/kpluginselector_p.h <span style="color: grey">(2dffae0)</span></li>
<li>staging/kcmutils/src/ksettings/componentsdialog.cpp <span style="color: grey">(111549a)</span></li>
<li>staging/kcmutils/src/ksettings/dialog.cpp <span style="color: grey">(131f75f)</span></li>
<li>staging/kcmutils/src/ksettings/dispatcher.cpp <span style="color: grey">(48bb0fc)</span></li>
<li>staging/kcmutils/src/ksettings/pluginpage.cpp <span style="color: grey">(0577908)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/113045/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>