On Mon, 6 Jun 2005 03:10:53 -0700, Yitzchak Scott-Thoennes <[EMAIL PROTECTED]> wrote:
> On Mon, Jun 06, 2005 at 11:49:17AM +0200, H.Merijn Brand wrote: > > On Mon, 6 Jun 2005 02:30:53 -0700, Yitzchak Scott-Thoennes > > <[EMAIL PROTECTED]> > > > - if (key <= AvFILLp(av) && AvARRAY(av)[key] != &PL_sv_undef > > > - && AvARRAY(av)[key]) > > > + if (key <= AvFILLp(av) && AvARRAY(av)[key] > > > + && (AvARRAY(av)[key] != &PL_sv_undef || ! AvREAL(av))) > > > > Could I get an extra set of parens somewhere here. It always feels icky to > > see a mixture of && and || in a line for a simple regex. If not only to > > prevent compilers that treat && and || exactly the same precedence > > Not sure where you mean. It's if ( a && b && ( c || ! d ) ) . I'm getting old :) -- H.Merijn Brand Amsterdam Perl Mongers (http://amsterdam.pm.org/) using Perl 5.6.2, 5.8.0, 5.8.5, & 5.9.2 on HP-UX 10.20, 11.00 & 11.11, AIX 4.3 & 5.2, SuSE 9.2 & 9.3, and Cygwin. http://www.cmve.net/~merijn Smoking perl: http://www.test-smoke.org, perl QA: http://qa.perl.org reports to: [EMAIL PROTECTED], perl-qa@perl.org
