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

Re: svn commit: r1488693 - /subversion/trunk/subversion/bindings/swig/perl/native/Client.pm

From: Daniel Shahaf <danielsh_at_apache.org>
Date: Sun, 7 Jul 2013 18:11:32 +0300

Roderich Schupp wrote on Sun, Jul 07, 2013 at 17:04:07 +0200:
> On Sun, Jul 7, 2013 at 3:14 PM, Daniel Shahaf <danielsh_at_apache.org> wrote:
>
> > > - POD: Use modern sub parameter conventions in POD (no more
> > "shift"ing).
> >
> > These all non-functional changes. We don't normally backport such changes
> >
>
> There's a difference between Subversion proper and the (Perl, Python, Ruby)
> bindings. The latter are useless without proper documentation,

Nothing is useful without proper documentation, but devil's advocate
says the bindings can be quite usable if you document only the delta
between SVN::Client->info42 and svn_client_info42().

> so IMHO the POD is "functional".

You changed:

sub {
  my $x = shift;
  my $y = shift;
}

to:

sub {
  my ($x, $y) = @_;
}

That is not a functional change. It is not visible to callers or to
documentation readers.
Received on 2013-07-07 17:11:50 CEST

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