fix rangematch('zz-')

[perl/list-index.git] / lib / List / Index.pm

diff --git a/lib/List/Index.pm b/lib/List/Index.pm
index 47ef0da75030c8cda582a1b1bf93d4b9a0db493d..3663c2459422638225058204dbc9854f667f315f 100644 (file)
--- a/lib/List/Index.pm
+++ b/lib/List/Index.pm
@@ -23,8 +23,8 @@ sub ranges {
        my $pages    = $options->{pages   } || 1 + int $#$self / $pagesize;
 
        $pagesize = @$self / $pages;
-       my $offset = 0;
-       my @links;
+       my $offset = $pagesize;
+       my @links = ('');
        while ($offset < @$self) {
                my $link = substr $self->[$offset], 0, $length;
                if ($context) {
@@ -57,19 +57,19 @@ sub ranges {
                        }
                }
 
-               push @links, [$link];
+               push @links, $link;
                $offset += $pagesize;
        }
 
+       use List::MoreUtils 'uniq';
+       @links = uniq @links;
        for my $i (0 .. $#links - 1) {
-               my ($link, $lastchar) = $links[$i + 1]->[0] =~ /(.*)(.)/;
-               $link .= $lastchar eq 'a' ? '.' : chr( ord($lastchar) - 1 )
-                       unless $lastchar eq '.';
-               next if $link eq $links[$i]->[0] and $i;
-               $links[$i]->[1] = $link;
+               my ($link, $lastchar) = $links[$i + 1] =~ /(.*)(.)/;
+               $link .= $lastchar le 'a' ? '.' : chr( ord($lastchar) - 1 );
+               next if $link eq $links[$i] and $i;
+               $links[$i] .= '-'.$link;
        }
-       $links[-1]->[1] = '';
-       $links[0]->[0] = '';
+       $links[-1] .= '-';
 
        return \@links;
 }
@@ -92,6 +92,7 @@ sub rangematch {
                        my $next = $char;
                        $next = chr( ord($char) + 1 ) if length $s1 > $i + 1;
                        my $last = 'z';
+                       next if $next gt $last;
                        if (length $s2 > $i) {
                                if ($s2 =~ /^\Q$prefix/) {
                                        $last = substr $s2, $i, 1;