On Tue, 15 May 2007, Kouhei Sutou wrote:
> 2007/5/15, Joe Swatosh <firstname.lastname@example.org>:
> >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.
Received on Tue May 15 02:37:10 2007
- application/pgp-signature attachment: stored