<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="https://git.reviewboard.kde.org/r/117185/">https://git.reviewboard.kde.org/r/117185/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Hmm. It looks like a bit of a kludge, but it works. No objections, if sebas and ervin agree as well.</pre>
<br />
<p>- David Faure</p>
<br />
<p>On March 30th, 2014, 11:28 a.m. UTC, Alex Merry wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://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 and David Faure.</div>
<div>By Alex Merry.</div>
<p style="color: grey;"><i>Updated March 30, 2014, 11:28 a.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
kservice
</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;">Remove KPluginLoader dependency on KService
The reason for this is so that KPluginLoader, KPluginFactory and
kexportplugin.h do not require anything else from the KService
framework, allowing them to be moved to KCoreAddons.
A KPluginName class is introduced to act as an intermediary between
KService and KPluginLoader. The KPluginLoader(KService) constructor is
replaced with KPluginLoader(KPluginName), and KService gains a cast
operator to convert it to a KPluginName.
Now
KService service("blah.desktop");
KPluginLoader(service);
works as before, maintaining source compatibility.
The reason for the intermediary class (rather than just, say, providing
a QString() operator to KService) is to make sure this combination is
the only one that works (and you can't start using KService everywhere a
QString is expected).</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </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;">Builds, tests pass.</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>autotests/CMakeLists.txt <span style="color: grey">(dcc4a40bd81f7e17f10219649214a8c9aac5cc60)</span></li>
<li>autotests/jsonplugin.h <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/jsonplugin.cpp <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/jsonplugin.json <span style="color: grey">(PRE-CREATION)</span></li>
<li>autotests/kpluginloadertest.cpp <span style="color: grey">(2b0afb81469f18f93ea43e6ac73df95a60cbd7f3)</span></li>
<li>autotests/kservicetest.h <span style="color: grey">(aa1c691c75c9e5b6903bcbf6e2dc95809ee1ce21)</span></li>
<li>autotests/kservicetest.cpp <span style="color: grey">(796abbcc6313d2cfd69bd213281ea30dfebaa421)</span></li>
<li>autotests/noplugin.desktop <span style="color: grey">(PRE-CREATION)</span></li>
<li>src/plugin/kpluginloader.h <span style="color: grey">(4e10edf0f152b409c6a4031610ff1f2f9771d08d)</span></li>
<li>src/plugin/kpluginloader.cpp <span style="color: grey">(1602c180db5e1a5f0765d95d68f90d2046c9ef2b)</span></li>
<li>src/services/kservice.h <span style="color: grey">(6df389a130587eea732b1d81718181bbd3df4f6d)</span></li>
<li>src/services/kservice.cpp <span style="color: grey">(c897d4c2a08d048e817154101b3a212a098f9768)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/117185/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>