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

Re: [PATCH] work in progress perl typemaps for swig

From: Josef Wolf <jw_at_raven.inka.de>
Date: 2003-07-17 23:02:57 CEST

On Tue, Jul 15, 2003 at 01:38:21PM +0800, Chia-liang Kao wrote:

> now here's some changes to make perl bindings work. you could run
> the following code now:

Fine :)

> my $pool = SVN::core::svn_pool_create(undef);
> my $repo = SVN::repos->svn_repos_open('/Users/clkao/svn/pairang', $pool);
> my $fs = SVN::repos::svn_repos_fs($repo);
> print SVN::fs::svn_fs_youngest_rev($fs, $pool);

Don't you think the pool handling stuff should be hidden? IMHO too much
details of the original API is shining through. Some additional abstraction
could lead to much more perlish code:

  my $repo=SVN::Repos->new("http://foo.bar/path");
  my @revs=$repo->revisions();
  print $revs[$#revs]->{revnumber}; # youngest revision
  print $revs[$#revs]->{log}; # log of the youngest revision
  # etc/pp

-- Josef Wolf -- jw@raven.inka.de --
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Jul 17 23:04:13 2003

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.