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

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

• Re: [perl #105278] Add warning for exists($a[0]) and delete($a[0]) by Eric Brine
• Re: [perl #105278] Add warning for exists($a[0]) and delete($a[0]) by Nicholas Clark
• Re: [perl #105278] Add warning for exists($a[0]) and delete($a[0]) by Eric Brine
• [perl #105278] Add warning for exists($a[0]) and delete($a[0]) by Father Chrysostomos via RT
• [perl #105278] Add warning for exists($a[0]) and delete($a[0]) by Eric Brine
• Re: [perl #105278] Add warning for exists($a[0]) and delete($a[0]) by Eric Brine
• Re: [perl #105278] Add warning for exists($a[0]) and delete($a[0]) by Eric Brine
• Re: [perl #105278] Add warning for exists($a[0]) and delete($a[0]) by Eric Brine
• Re: [perl #105278] Add warning for exists($a[0]) and delete($a[0]) by Jesse Luehrs
• Re: [perl #105278] Add warning for exists($a[0]) and delete($a[0]) by Jesse Luehrs
• Re: [perl #105278] Add warning for exists($a[0]) and delete($a[0]) by Ricardo Signes
• Re: [perl #105278] Add warning for exists($a[0]) and delete($a[0]) by Jesse Luehrs
• [perl #105278] Add warning for exists($a[0]) and delete($a[0]) by Father Chrysostomos via RT