diff options
| author | Eric Wong <e@80x24.org> | 2016-10-05 23:47:28 +0000 |
|---|---|---|
| committer | Eric Wong <e@80x24.org> | 2016-10-05 23:53:35 +0000 |
| commit | 3c9dd6619f825f0515e7e4afa1bd55c99c1a68d3 (patch) | |
| tree | 60c997b670ccd6b7a1fc10aa46232f86afa4e3da /lib/PublicInbox/SearchView.pm | |
| parent | 728d8dbcb920cd19f2fe4d7479d770c540274690 (diff) | |
| download | public-inbox-3c9dd6619f825f0515e7e4afa1bd55c99c1a68d3.tar.gz | |
This bug was hidden, and we may not be able to efficiently implement a topmost subroutine with the hash-based (vs linked-list) based container for threading in the next commit.
Diffstat (limited to 'lib/PublicInbox/SearchView.pm')
| -rw-r--r-- | lib/PublicInbox/SearchView.pm | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/lib/PublicInbox/SearchView.pm b/lib/PublicInbox/SearchView.pm index ebeb41f7..cfe6dfff 100644 --- a/lib/PublicInbox/SearchView.pm +++ b/lib/PublicInbox/SearchView.pm @@ -155,9 +155,8 @@ sub mset_thread { $th->thread; if ($q->{r}) { # order by relevance $th->order(sub { - [ sort { (eval { $pct{$b->topmost->{id}} } || 0) - <=> - (eval { $pct{$a->topmost->{id}} } || 0) + [ sort { ( $pct{$b->{id}} || 0) <=> + ( $pct{$a->{id}} || 0) } @{$_[0]} ]; }); } else { # order by time (default for threaded view) |