Let kquitapp fail verbosely

Stefan Monov logixoul at gmail.com
Sun May 25 07:13:58 BST 2008

Hi, thanks for the comments. :)

> Since KDE includes are already present I'd use kError() or kWarning() (not
> kDebug() either way).
kWarning() outputs:
kquitapp(4511)/ main: "foo: no application quit"
Quite ugly and debuggish. At least, the function "main" and the
quotation marks shouldn't be there. Is kWarning() really intended for
use with command line tools?

> Also in theory people can have a notification pop up
> for these types of events if there's not running kquitapp from the shell.
But this popup would be annoying if I actually worked in the shell,
and I don't think kquitapp is told who launched it.

I attached the current patch for reference.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: kquitapp.diff
Type: text/x-patch
Size: 1017 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20080525/70f8c013/attachment.bin>

More information about the kde-core-devel mailing list