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

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