[Date Prev][Date Next]
[Chronological]
[Thread]
[Top]
Re: commit: ldap/servers/slapd/back-perl SampleLDAP.pm
- To: Gavin Henry <ghenry@suretecsystems.com>
- Subject: Re: commit: ldap/servers/slapd/back-perl SampleLDAP.pm
- From: Hallvard B Furuseth <h.b.furuseth@usit.uio.no>
- Date: Sun, 1 Jul 2007 14:46:48 +0200
- Cc: openldap-devel@openldap.org
- In-reply-to: <hbf.20070615oltz@bombur.uio.no>
- References: <200706151125.l5FBPW1Q046796@cantor.openldap.org> <42433.212.159.59.85.1181911607.squirrel@webmail.suretecsystems.com> <hbf.20070615oltz@bombur.uio.no>
I wrote:
>Gavin Henry writes:
>> I've run Perl::Critic over this and fixed/updated to latest "Perl Best
>> Practices". We might as well have an up to date sample.
>>
>> Do you mind if I commit my changes?
>
> My impression is that back-perl hovers on the verge of code rot, and any
> change is likely for the better.
Er... not this one, though:-( Our Perl version (5.8.8) interprets
print <STDERR>, "Here in new\n";
as "read data from STDERR, then print that and the argument string to
STDOUT". Is this some bleeding edge change to Perl?
Also I think the initial 'package' and 'use' statements should go below
the comment at the top, and I'm not sure how new the "our" keyword is.
I suspect there are some pretty old Perl versions out there, in
particular on Windows.
--
Hallvard