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

Re: svn_error_purge_tracing() and clearing the errors Re: svn commit: r1051763 - in /subversion/trunk/subversion: include/private/svn_repos_private.h libsvn_repos/commit.c mod_dav_svn/version.c

From: Daniel Shahaf <d.s_at_daniel.shahaf.name>
Date: Fri, 24 Dec 2010 11:32:49 +0200

Blair Zajac wrote on Thu, Dec 23, 2010 at 17:09:31 -0800:
> On 12/23/10 3:01 AM, Daniel Shahaf wrote:
>> I've made a quick sketch --- looks good?
> The suggestions look good.
> We could add some more tests, checking the behavior with SVN_NO_ERROR
> input and one where all errors are tracing, which should return an error
> from SVN_ERR_ASSERT().

I'm not sure about the 'assert' part (both due to the "testing that it
has asserted" issue and due to the fact that it's mathematiclaly
legitimaste to return an empty chain (aka SVN_NO_ERROR) in that case).

As to the rest, feel free to commit some tests (whether based on my
sketch or not); I'd rather not block you, and it might be a little while
before I can get to it myself...

> As an aside, there's a comment about adding an svn_boolean_t to
> svn_error_t instead of using strcmp(), which could be done in a totally
> separate commit.
Received on 2010-12-24 10:35:51 CET

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.