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

Re: svn commit: r1052817 - /subversion/trunk/subversion/tests/libsvn_subr/error-test.c

From: <kamesh.jayachandran_at_gmail.com>
Date: Sun, 26 Dec 2010 09:24:58 +0000

As at HEAD(r1052844) i could still see that docstring in subversion/include/svn_error.h

----------
Sent via Nokia Email

------Original message------
From: Daniel Shahaf <d.s_at_daniel.shahaf.name>
To: "Kamesh Jayachandran" <kamesh_at_collab.net>
Cc: <dev_at_subversion.apache.org>,<commits_at_subversion.apache.org>
Date: Sunday, December 26, 2010 10:59:43 AM GMT+0200
Subject: Re: svn commit: r1052817 - /subversion/trunk/subversion/tests/libsvn_subr/error-test.c

You're quoting an old version of the docstring.

Kamesh Jayachandran wrote on Sun, Dec 26, 2010 at 12:10:46 +0530:
> >+
> >+ err2 = svn_error_purge_tracing(err);
> >+ for (child = err2; child; child = child->child)
> >+ if (svn_error__is_tracing_link(child))
> >+ {
> >+ return svn_error_create(SVN_ERR_TEST_FAILED, err,
> >+ "Tracing link found after purging the "
> >+ "following chain:");
> >+ }
> >+
> >+ svn_error_clear(err);
>
> The doc of svn_error_purge_tracing(err) says err should *not* be cleared.
>
> <snip>
> /**
> * Purge from @a err and its child chain any links associated with
> * error tracing placeholders, and return the new top-level error
> * chain item. @a err should be considered unusable after passing
> * through this function, but should *not* be cleared (as the returned
> * error shares memory with @a err). @a err can be #SVN_NO_ERROR.
> *
> * @since New in 1.7.
> */
> svn_error_t *svn_error_purge_tracing(svn_error_t *err);
> </snip>
>
> Are you sure?
>
> With regards
> Kamesh Jayachandran
>
>
Received on 2010-12-26 12:23:30 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.