You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Lieven Govaerts <sv...@mobsol.be> on 2008/07/30 21:17:49 UTC

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

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

arfrever@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@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org