Revert "pagespec_match_list * optimisation"
This reverts commit 2f96c49bd1
.
I forgot about internal pages. We don't want * matching them!
I left the optimisation in pagecount, where it used to live.
Internal pages probably don't matter when they're just being
counted.
master
parent
4030f75765
commit
85ae48b21e
|
@ -1837,8 +1837,6 @@ sub pagespec_match_list ($$;@) {
|
||||||
my $spec=shift;
|
my $spec=shift;
|
||||||
my @params=@_;
|
my @params=@_;
|
||||||
|
|
||||||
return @$pages if $spec eq '*'; # optimisation
|
|
||||||
|
|
||||||
my $sub=pagespec_translate($spec);
|
my $sub=pagespec_translate($spec);
|
||||||
error "syntax error in pagespec \"$spec\""
|
error "syntax error in pagespec \"$spec\""
|
||||||
if $@ || ! defined $sub;
|
if $@ || ! defined $sub;
|
||||||
|
|
|
@ -26,7 +26,9 @@ sub preprocess (@) {
|
||||||
# register a dependency.
|
# register a dependency.
|
||||||
add_depends($params{page}, $params{pages});
|
add_depends($params{page}, $params{pages});
|
||||||
|
|
||||||
my @pages=pagespec_match_list([keys %pagesources], $params{pages}, location => $params{page});
|
my @pages=keys %pagesources;
|
||||||
|
@pages=pagespec_match_list(\@pages, $params{pages}, location => $params{page})
|
||||||
|
if $params{pages} ne "*"; # optimisation;
|
||||||
return $#pages+1;
|
return $#pages+1;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue