develooper Front page | perl.perl5.porters | Postings from July 2012

Re: [perl #105278] Add warning for exists($a[0]) and delete($a[0])

Thread Previous | Thread Next
From:
Jesse Luehrs
Date:
July 2, 2012 23:16
Subject:
Re: [perl #105278] Add warning for exists($a[0]) and delete($a[0])
Message ID:
20120703061605.GK30375@tozt.net
On Sat, Dec 31, 2011 at 04:17:44AM -0600, Jesse Luehrs wrote:
> On Sat, Dec 31, 2011 at 01:12:16AM -0500, Eric Brine wrote:
> > On Sat, Dec 31, 2011 at 12:53 AM, Father Chrysostomos via RT <
> > perlbug-followup@perl.org> wrote:
> > 
> > > On Fri Dec 16 11:26:15 2011, ikegami@adaelis.com wrote:
> > > > Hi,
> > > >
> > > > The text of the opening post (below) describes patches #2 and #3.
> > > >
> > > > Patch #1 changes two C<< exist($_[1]) >> to C<< @_ >= 2 >>. Any
> > > > objections
> > > > to that? They aren't part of tests for exists($a[]) or delete($a[]).
> > >
> > > I actually think the code reads more clearly with the exists() form.
> > >
> > 
> > But it's not valid code according to the documentation. Are you saying the
> > documentation should be changed?
> 
> The documentation says that it is deprecated, not invalid. That said, I
> think it being deprecated is enough reason to agree with the patch.

And patch #1 is now applied as 5e5bb7c. Leaving #2 and #3 for now until
we come to a consensus on this issue.

-doy

Thread Previous | Thread Next


nntp.perl.org: Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at ask@perl.org | Group listing | About