Adjustments for signal handling

Thiago Macieira thiago at kde.org
Thu Nov 27 10:29:24 GMT 2008


Markus Elfring wrote:
>Thiago Macieira schrieb:
>> On the specific cases where the handlers are really needed, we should
>> do them properly.
>
>What should be done with the following update candidates?
>
>Bug #156560: signal_exit ...
>http://websvn.kde.org/trunk/KDE/kdebase/runtime/kdesu/kdesud/kdesud.cpp?
>revision=720106&view=markup

All of the signals here already quit the application. The only reason for 
having this handler is to print a message.

Pretty useless, I'd say. So remove it.

>Bug #156562: sigAlarm ...
>http://websvn.kde.org/trunk/KDE/kdebase/workspace/kdm/kfrontend/kgapp.cp
>p?revision=861292&view=markup

Oswald, can you comment on this one? There's a timer and an alarm in this 
app. Why both?

-- 
  Thiago Macieira  -  thiago (AT) macieira.info - thiago (AT) kde.org
    PGP/GPG: 0x6EF45358; fingerprint:
    E067 918B B660 DBD1 105C  966C 33F5 F005 6EF4 5358
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 189 bytes
Desc: This is a digitally signed message part.
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20081127/e8e0dd78/attachment.sig>


More information about the kde-core-devel mailing list