Re: svn commit: r25517 - in trunk/subversion: include libsvn_client

On 7/6/07, Karl Fogel <kfogel@red-bean.com> wrote:
> I wrote:
> > "David Glasser" <glasser@mit.edu> writes:
> > >> The reason why I added a second macro is to make it clear to users of
> > >> the SVN_DEPTH_FROM_RECURSE macro that it doesn't give the correct
> > >> results in all cases, more specifically when working with status.
> > >
> > > In fact it ought to be used in more places, like
> > > svnserve/serve.c(status)... (and I think somewhere in mod_dav_svn as
> > > well)
> >
> > I believe the status command is special and doesn't do the same
> > recurse->depth mappings as other commands, and that's why it doesn't
> > use the macro.
> >
> > Checking... yup: it maps nonrecursive to svn_depth_empty instead of
> > svn_depth_files.
>
> Oh, sorry, I see you were actaully talking about the
> SVN_DEPTH_FROM_RECURSE_STATUS macro, it just didn't happen to be
> quoted in the context, so mistakenly read "it" as referring to
> SVN_DEPTH_FROM_RECURSE.

Yup, over-zealous context pruning on my part.

--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