Re: [PATCH] issue #897: libsvn_client

Erik Huelsmann wrote:

>Hmm, Now that you mention those. I think I should have indeed picked up on
>that. I will revise the patch and resubmit when it is ready.
>
>One problem remains: I raised the question on IRC, bug Greg Hudson said it
>should not be a problem, since there are no long error messages: the issue
>discusses wrapping at 70 columns. We do have long error messages (when filenames
>are %s-ed in) and we do have messages with multiple lines. Is it an option
>to svn_error_quick_wrap the extra lines into other error structures?
>
>
If we know that the message has several "logical" lines (as in the
checksum mismatch example), then IMHO we should use
svn_error_quick_wrap. Any other kind of wrapping sould have to be done
by the client, since the client knows best how to display the errors.
Yes, the cmdline-specific svn_handle_error is in libsvn_subr, but that's
a special case and GUI clients can do anything they like.

-- 
Brane Čibej   <brane_at_xbc.nu>   http://www.xbc.nu/brane/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org