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

On Tue, 15 May 2007, Kouhei Sutou wrote:

> Hi,
>
> 2007/5/15, Joe Swatosh <joe.swatosh@gmail.com>:
>
> >[[[
> >Follow on to r25007 which added depth support to svn_client_proplist3.
> >
> >* subversion/bindings/swig/ruby/svn/client.rb
> >(Svn::Client::Context#proplist) changed argument name and default to
> >reflect the use of depth instead of recurse in the core API.
> >]]]
> >
> >Index: subversion/bindings/swig/ruby/svn/client.rb
> >===================================================================
> >--- subversion/bindings/swig/ruby/svn/client.rb (revision 25014)
> >+++ subversion/bindings/swig/ruby/svn/client.rb (working copy)
> >@@ -215,7 +215,7 @@
> > # Returns list of properties attached to +target+ as an Array of
> > # Svn::Client::PropListItem.
> > # Paths and URIs are available as +target+.
> >- 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.