<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0//EN" "http://www.w3.org/TR/REC-html40/strict.dtd"><html><head><meta name="qrichtext" content="1" /><style type="text/css">p, li { white-space: pre-wrap; }</style></head><body style=" font-family:'Consolas'; font-size:11pt; font-weight:400; font-style:normal;">On Sunday 08 February 2009, Thiago Macieira wrote:<br>
> Oswald Buddenhagen wrote:<br>
> >On Sun, Feb 08, 2009 at 12:31:27PM +0000, Dario Freddi wrote:<br>
> >> SVN commit 923137 by dafre:<br>
> >><br>
> >> Adding a signal to ksmserver, that notifies DBus when KDE is about to<br>
> >> logout.<br>
> ><br>
> >errrm ... how about using standard interfaces and hooking into session<br>
> >management instead? see the qt doc for starters.<br>
><br>
> Not all applications are graphical and connect to X11. In KDE3 we had this<br>
> signal in DCOP anyways.<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p>I would say that it's poor form to up and make changes to such important KDE internal code as ksmserver without at least running it by kde-core-devel beforehand (at least not until you've accumulated a lot of experience in the code).  Just something for Dario and others to keep in mind in the future. :)<br>
<br />Regards,<br>
 - Michael Pyne<br>
<p style="-qt-paragraph-type:empty; margin-top:0px; margin-bottom:0px; margin-left:0px; margin-right:0px; -qt-block-indent:0; text-indent:0px; -qt-user-state:0;"><br></p></body></html>