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

Re: svn commit: r32337 - trunk/subversion/libsvn_client

From: Lieven Govaerts <svnlgo_at_mobsol.be>
Date: Wed, 30 Jul 2008 23:17:49 +0200

Reverted this change because it broke 2 tests, this was clearly not your
intended behavior.

arfrever_at_tigris.org wrote:
> Author: arfrever
> Date: Wed Jul 30 11:37:26 2008
> New Revision: 32337
>
> Log:
> Follow-up to r32316:
>
> * subversion/libsvn_client/cat.c
> (svn_client_cat2): Code simplification. No functional change.
>
> Modified:
> trunk/subversion/libsvn_client/cat.c
>
> Modified: trunk/subversion/libsvn_client/cat.c
> URL: http://svn.collab.net/viewvc/svn/trunk/subversion/libsvn_client/cat.c?pathrev=32337&r1=32336&r2=32337
> ==============================================================================
> --- trunk/subversion/libsvn_client/cat.c Wed Jul 30 08:12:50 2008 (r32336)
> +++ trunk/subversion/libsvn_client/cat.c Wed Jul 30 11:37:26 2008 (r32337)
> @@ -165,17 +165,15 @@ svn_client_cat2(svn_stream_t *out,
> const char *url;
> svn_stream_t *output = out;
>
> + peg_revision = svn_cl__rev_default_to_head_or_working(peg_revision,
> + path_or_url);
> /* ### Inconsistent default revision logic in this command. */
> if (peg_revision->kind == svn_opt_revision_unspecified)
> {
>
                   ^^^^^^^^^
The original peg revision, which is now overwritten, was still used here.

Lieven

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-07-30 23:18:37 CEST

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.