[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

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

From: Husterer, Thomas RD-CP1 <Thomas.Husterer_at_heidelberg.com>
Date: 2006-03-21 16:10:17 CET

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
Received on Fri Mar 24 13:10:53 2006

This is an archived mail posted to the Subversion Users mailing list.