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

[David Faure]

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

Ship it!


Thanks!


kioclient/kioclient.cpp
<http://git.reviewboard.kde.org/r/110964/#comment25143>

    The %2/%1 order is a bit surprising, but ok, why not :-)


- David Faure


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/20130612/1c043f29/attachment.htm>