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

Re: svn commit: r22757 - branches/1.4.x

From: David Glasser <glasser_at_mit.edu>
Date: 2006-12-20 05:47:16 CET

On 12/19/06, dlr@tigris.org <dlr@tigris.org> wrote:

> --- branches/1.4.x/STATUS (original)
> +++ branches/1.4.x/STATUS Tue Dec 19 13:41:28 2006
> @@ -91,6 +91,7 @@
> Votes:
> +1: glasser
> +1: clkao (tested but vote not counted)
> + -0: dlr (how does r22614 figure into this?)

The nominated revision fixes the actual bug (function does not work on
transaction roots). It uses an ugly but necessary idiom that is used
in a couple other places as well.

r22610 introduces a new API (which I assume shouldn't be backported)
which eliminates the need to use that idiom; r22614 and r22629 clean
up the code in a way that only works with the new API.

I believe that backporting r22609 alone is a reasonable choice.

--dave

-- 
David Glasser | glasser_at_mit.edu | http://www.davidglasser.net/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Dec 20 05:47:28 2006

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