G'day hurricup,
++ Good catch!
The code itself parses as legal Perl:
$ perl -MO=Deparse,-p -e 'my $l = ref $a eq "ARRAY" ? ($a->[0] || $->[
+1]) : $a;'
(my $l = ((ref($a) eq 'ARRAY') ? ($a->[0] || ($- > [1])) : $a));
-e syntax OK
And, as you can see, your analysis is correct: '$-' is being compared with '[1]'.
There's actually two adjacent lines like that in the source code:
# Deal with the case of an item actually being an array ref to 1 o
+r 2
# hashrefs. Don't assign to $a or $b, as they're aliases to the or
+ignal
my $l = ref $a eq 'ARRAY' ? ($a->[0] || $->[1]) : $a;
my $r = ref $b eq 'ARRAY' ? ($b->[0] || $->[1]) : $b;
I suspect those should be:
my $l = ref $a eq 'ARRAY' ? ($a->[0] || $a->[1]) : $a;
my $r = ref $b eq 'ARRAY' ? ($b->[0] || $b->[1]) : $b;
although, I haven't delved deeply enough into the code to say for certain.
I suggest you raise a bug report for this.
-
Are you posting in the right place? Check out Where do I post X? to know for sure.
-
Posts may use any of the Perl Monks Approved HTML tags. Currently these include the following:
<code> <a> <b> <big>
<blockquote> <br /> <dd>
<dl> <dt> <em> <font>
<h1> <h2> <h3> <h4>
<h5> <h6> <hr /> <i>
<li> <nbsp> <ol> <p>
<small> <strike> <strong>
<sub> <sup> <table>
<td> <th> <tr> <tt>
<u> <ul>
-
Snippets of code should be wrapped in
<code> tags not
<pre> tags. In fact, <pre>
tags should generally be avoided. If they must
be used, extreme care should be
taken to ensure that their contents do not
have long lines (<70 chars), in order to prevent
horizontal scrolling (and possible janitor
intervention).
-
Want more info? How to link
or How to display code and escape characters
are good places to start.