[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r1481628 - /subversion/trunk/subversion/libsvn_repos/commit.c

From: Daniel Shahaf <d.s_at_daniel.shahaf.name>
Date: Mon, 13 May 2013 04:44:29 +0300

Bert Huijben wrote on Sun, May 12, 2013 at 22:48:37 +0200:
> In this specific case I'm pretty sure that the error will actually
> just go to the user, but in general I would be -0.5 on performing this
> kind of escapes in the creation of error messages that are part of our
> libraries that are also used by GUIs and bindings that might want to
> apply different handling and see the raw values.

Are you suggesting we need to support a way to _parse_ the path out of
the error message? If so, perhaps we should add a 'const char *extra[];'
member to struct svn_error_t.

If we extend svn_error_t we might also look into supporting >1 children,
to facilitate svn_error_compose_create(); compare Python's "raise from".

Daniel
(based on a conversation I had with Mattias yesterday)
Received on 2013-05-13 03:45:15 CEST

This is an archived mail posted to the Subversion Dev mailing list.