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. -doyThread Previous | Thread Next