You are viewing a plain text version of this content. The canonical link for it is here.
Posted to users@subversion.apache.org by "Husterer, Thomas RD-CP1" <Th...@heidelberg.com> on 2006/03/21 15:10:17 UTC

possible bug in ruby-binding Svn::Client::Context@info

the svn-ruby binding contains a wrapper function
Svn::Client::Context@info 
which calls Client.info() in file client.rb (line 264).

      def info(path_or_uri, rev=nil, peg_rev=nil, recurse=false)
        rev ||= URI(path_or_uri).scheme ? "HEAD" : "BASE"
        peg_rev ||= rev
        receiver = Proc.new do |path, info|
          yield(path, info)
        end
        Client.info(path_or_uri, rev, peg_rev, receiver, recurse, self)
      end

If you look at the c-header file svn_client.h then it seems that the two

parameters 'rev' and 'peg_rev' are swapped.
This is no large problem because the swap is consequently 
done in the definition and in the call to Client.info in the body of the

wrapper function.

The only effect is a bad handling of the default values and 
the bad documenting effect.

Anyway it would be nice to fix this little typo in one of the next
releases.

thanks Thomas