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

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.

-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