<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/101316/">http://git.reviewboard.kde.org/r/101316/</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;">what i like about it: it would make switching from plasma-desktop to plasma-netbook a bit easier from the kcm that enables that; it would make it more likethe windowmanager and reduce instructions on how to restart plasma-dekstop to one step.
what concerns me is the crashes you note. given that it is the weather engine, my guess is that the global objects are not being cleaned up before the applets. (so the dataengine is being destroyed before the weather applet is gone.) i can't see anything in the patch that should change those semantics though; i assume that PlasmaApp::cleanup() is still getting called before ~PlasmaApp? perhaps you could throw a backtrace somewhere to look at.
my other concern is that this makes plasma-desktop rely on X11. what happens when we move to a different system, e.g. wayland?</pre>
<br />
<p>- Aaron J.</p>
<br />
<p>On May 8th, 2011, 7:43 p.m., Martin Gräßlin wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Plasma.</div>
<div>By Martin Gräßlin.</div>
<p style="color: grey;"><i>Updated May 8, 2011, 7:43 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;">Replaces KUniqueApplication by a KApplication and adds a selection owner for Plasma Shells like the selection owner used by KWin (WMs in general). If plasma-desktop is already running, it is possible to restart with "plasma-desktop --replace". If plasma-desktop is running and you don't add the --replace, it will only yell (like KWin). The same could be added to the other shells like netbook, to easily restart/switch during development. I got the idea from this thread on wm-spec mailinglist: http://mail.gnome.org/archives/wm-spec-list/2011-January/msg00002.html
Current state: it works, plasma-desktop is replaced, but sometimes the WeatherEngine is causing a crash when exiting the previous one. So it is possible that the change from KUniqueApplication to KApplication is opening Pandora's Box. You decide whether it is worth to spend more time on it.</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>plasma/desktop/shell/main.cpp <span style="color: grey">(6f04e98)</span></li>
<li>plasma/desktop/shell/plasmaapp.h <span style="color: grey">(06ee523)</span></li>
<li>plasma/desktop/shell/plasmaapp.cpp <span style="color: grey">(58e8947)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/101316/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>