<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/110482/">http://git.reviewboard.kde.org/r/110482/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On May 21st, 2013, 10:46 a.m. UTC, <b>Kevin Ottens</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;">I think it's taking the right direction. Probably good to get rid of ki18n if we don't use any of its advanced features (I don't think we do). As for kwidgets and kwidgetsaddons, it depends a bit on the aim we have for that library and which tier we want for it. If we aim tier1 it can't depend on them, if we aim tier2 it can depend on kwidgetsaddons, if we aim tier3 it can depend on both.
Any idea why it requires KWidgets and KWidgetsAddons?
Test should be streamlined to bring less dependencies too.</pre>
</blockquote>
<p>On May 21st, 2013, 11:18 a.m. UTC, <b>Marco Martin</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;">i think kwidgets and kwidgetsaddons are required by the "are sure you want to quit" dialog, shar should be rebuilt using only basic qt stuff.
another source of dependencies is the small test app, that uses kcmdlineargs and kiconloader for "findMovie"</pre>
</blockquote>
<p>On May 21st, 2013, 11:53 a.m. UTC, <b>Marco Martin</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;">ah, in general, what should replace KActionCollections?</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;">Well, there's no real replacement for KActionCollection, it's here to stay, but will be in XMLGUI which is a high tier framework. Going for a QHash as you did is likely a good thing for the status notifier, it doesn't need XMLGUI per say... Maybe we'll need conversion convenience to and from KActionCollection in XMLGUI though.
KCmdLineArgs can be replaced with QCommandLineParser (part of kdeqt5staging until it's in Qt itself).</pre>
<br />
<p>- Kevin</p>
<br />
<p>On May 21st, 2013, 12:45 p.m. UTC, Marco Martin 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 Marco Martin.</div>
<p style="color: grey;"><i>Updated May 21, 2013, 12:45 p.m.</i></p>
<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;">This implemets a step in the kdeui crumble epic.
moves the classes kstatusnotifieritem and knotificationsrestrictions in the knotifiactions library.
The patch works, but there are still several issues:
* porting from kdebug to qdebug loses the area number
* adds some link libraries: the classes add ki18n, kwidgets and KWidgetsAddons
* the test adds ki18n kde4support kdecore
* the KActionCollection becomes a qhash of actions: how should be kactioncollections ported?
I guess it should use the qt translation system, and redo the quit dialog to not usekstandardgui at all?</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>kdeui/CMakeLists.txt <span style="color: grey">(cfa29ef)</span></li>
<li>kdeui/notifications/knotificationrestrictions.h <span style="color: grey">(7b83d25)</span></li>
<li>kdeui/notifications/knotificationrestrictions.cpp <span style="color: grey">(a396fd6)</span></li>
<li>kdeui/notifications/kstatusnotifieritem.h <span style="color: grey">(be21882)</span></li>
<li>kdeui/notifications/kstatusnotifieritem.cpp <span style="color: grey">(37abe7e)</span></li>
<li>kdeui/notifications/kstatusnotifieritemdbus_p.h <span style="color: grey">(063343c)</span></li>
<li>kdeui/notifications/kstatusnotifieritemdbus_p.cpp <span style="color: grey">(6c9e1da)</span></li>
<li>kdeui/notifications/kstatusnotifieritemprivate_p.h <span style="color: grey">(32e7906)</span></li>
<li>kdeui/notifications/org.freedesktop.Notifications.xml <span style="color: grey">(8ddb421)</span></li>
<li>kdeui/notifications/org.kde.StatusNotifierItem.xml <span style="color: grey">(82c427d)</span></li>
<li>kdeui/notifications/org.kde.StatusNotifierWatcher.xml <span style="color: grey">(dd45dc9)</span></li>
<li>kdeui/tests/CMakeLists.txt <span style="color: grey">(cd055d5)</span></li>
<li>kdeui/tests/kstatusnotifieritemtest.h <span style="color: grey">(38a1e1a)</span></li>
<li>kdeui/tests/kstatusnotifieritemtest.cpp <span style="color: grey">(38e85ac)</span></li>
<li>staging/knotifications/src/CMakeLists.txt <span style="color: grey">(266b67c)</span></li>
<li>staging/knotifications/src/config-knotifications.h.cmake <span style="color: grey">(PRE-CREATION)</span></li>
<li>staging/knotifications/tests/CMakeLists.txt <span style="color: grey">(2240a69)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/110482/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>