Review Request 110964: kioclient: error messages do not identify their source

Commit Hook null at kde.org
Thu Jun 13 10:37:55 BST 2013


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/110964/#review34274
-----------------------------------------------------------


This review has been submitted with commit 14dd94842469ed7995cef6ffc15461406fc25dfd by Jonathan Marten to branch master.

- Commit Hook


On June 12, 2013, 12:15 p.m., Jonathan Marten wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/110964/
> -----------------------------------------------------------
> 
> (Updated June 12, 2013, 12:15 p.m.)
> 
> 
> Review request for KDE Runtime.
> 
> 
> Description
> -------
> 
> While investigating another problem (which turned out to be not the fault of KDE at all), I encountered this mysterious message on stderr:
> 
> Syntax Error: Not enough arguments
> 
> It took a long time to track down the source of this message and find out that it was from kioclient.  If kioclient had given its name in the error message, as most Unix commands do, then the source of the message would have been obvious and made investigation easier.
> 
> 
> Diffs
> -----
> 
>   kioclient/kioclient.cpp d879980 
> 
> Diff: http://git.reviewboard.kde.org/r/110964/diff/
> 
> 
> Testing
> -------
> 
> Built kioclient with these changes, verified error message output.
> 
> $ kioclient exec
> kioclient: Syntax error, not enough arguments
> $ kioclient exec a b c
> kioclient: Syntax error, too many arguments
> $ kioclient foo
> kioclient: Syntax error, unknown command 'foo'
> $
> 
> 
> Thanks,
> 
> Jonathan Marten
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20130613/f7e377c4/attachment.htm>


More information about the kde-core-devel mailing list