Re: [patch] ruby bindings tracking the recurse=>depth changes in svn_client_proplist3

Hi,

2007/5/15, Daniel Rall <dlr@collab.net>:
> > >- def proplist(target, rev=nil, peg_rev=nil, recurse=true, &block)
> > >+ def proplist(target, rev=nil, peg_rev=nil,
> > >depth=Core::DEPTH_INFINITY, &block)
> >
> > We can use depth=nil for the same propose. (see
> > svn_swig_rb_to_depth()) Could you use nil instead of
> > Svn::Core::DEPTH_INFINITY and commit this patch?
>
> FWIW, I like the use of an explicit value, as it makes the code more
> easy to understand.

We use depth=nil as default value in other methods.
depth=nil means "we use default depth value" and the default
depth value is used all depth parameter. I think using depth=nil
for all methods is more understandable rather than using
depth=Core::DEPTH_INFINITY only for Svn::Client#proplist.

Thanks,

--
kou
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org