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
> - Client.proplist3(target, rev, peg_rev, recurse, receiver, self)
> + Client.proplist3(target, rev, peg_rev, depth, receiver, self)
> alias prop_list proplist
This patch looks good, +1 to commit.
Received on Tue May 15 01:19:04 2007
- application/pgp-signature attachment: stored