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

On Mon, 14 May 2007, Joe Swatosh wrote:

> Do we want to try to keep up with the recurse to depth changes?

Yup -- they're not subject to as much flux as the MT changes.

> [[[
> 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)
> rev ||= "HEAD"
> peg_rev ||= rev
> items = []
> @@ -223,7 +223,7 @@
> items << PropListItem.new(path, prop_hash)
> block.call(path, prop_hash) if block
> end
> - Client.proplist3(target, rev, peg_rev, recurse, receiver, self)
> + Client.proplist3(target, rev, peg_rev, depth, receiver, self)
> items
> end
> alias prop_list proplist

This patch looks good, +1 to commit.