diff options
author | Karl Williamson <khw@cpan.org> | 2017-03-21 21:41:34 -0600 |
---|---|---|
committer | Karl Williamson <khw@cpan.org> | 2017-06-01 07:05:15 -0600 |
commit | 704b574dcdee24984d4fe89fd31a01cb6fc2ac72 (patch) | |
tree | 36aaaa3587389bdd8791298aef669333accb5bb0 /t/harness | |
parent | eb7486de5e577623ae719baf1b442bde992cb7c7 (diff) | |
download | perl-704b574dcdee24984d4fe89fd31a01cb6fc2ac72.tar.gz |
t/harness: Remove useless sort
Instead move its effect to the sort that overrides the first one. This
is because the tests are executed in the order of the rules to
TAP::Harness, not in the order of the test list.
Diffstat (limited to 't/harness')
-rw-r--r-- | t/harness | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -145,7 +145,7 @@ if (@ARGV) { push @seq, $next; my @last; - push @last, sort { lc $a cmp lc $b } + push @last, _tests_from_manifest($Config{extensions}, $Config{known_extensions}); my %times; if ($state) { @@ -174,7 +174,7 @@ if (@ARGV) { # sequentially. push @seq, { par => [ map { s!/$!/*!; { seq => $_ } } sort { # Directories, ordered by total time descending then name ascending - $total_time{$b} <=> $total_time{$a} || $a cmp $b + $total_time{$b} <=> $total_time{$a} || lc $a cmp lc $b } keys %dir ] }; $rules = { seq => \@seq }; |