diff options
| author | Eric Wong <e@80x24.org> | 2022-08-29 09:26:38 +0000 |
|---|---|---|
| committer | Eric Wong <e@80x24.org> | 2022-08-29 19:05:49 +0000 |
| commit | be55542acc90572da54d828313c2416ae1c902b1 (patch) | |
| tree | 001d9a6f38a16a023bd6f6b7a32a30403442a1ca /lib/PublicInbox/LeiSavedSearch.pm | |
| parent | a66a2eddda8ffbb7679f4ef80dba7c7d377cda84 (diff) | |
| download | public-inbox-be55542acc90572da54d828313c2416ae1c902b1.tar.gz | |
treewide: ditch inbox->recent method
It's a needless wrapper, nowadays. Originally, ->over was added
on experimental basis to optimize for /$INBOX/ where Xapian
->search is slower on gigantic (LKML-sized) inboxes.
Nowadays with extindex, ->over is here to stay given NNTP and
IMAP both benefit from it. So reduce the interpreter stack
overhead and just access ->over directly.
lxs->recent was never used outside of tests, anyways.
And while we're in the area, avoid needlessly bumping the
refcount of $ctx->{ibx} in View::paginate_recent.
Diffstat (limited to 'lib/PublicInbox/LeiSavedSearch.pm')
| -rw-r--r-- | lib/PublicInbox/LeiSavedSearch.pm | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/lib/PublicInbox/LeiSavedSearch.pm b/lib/PublicInbox/LeiSavedSearch.pm index 1d13aef6..ed92bfd1 100644 --- a/lib/PublicInbox/LeiSavedSearch.pm +++ b/lib/PublicInbox/LeiSavedSearch.pm @@ -299,7 +299,6 @@ no warnings 'once'; *smsg_by_mid = \&PublicInbox::Inbox::smsg_by_mid; *msg_by_mid = \&PublicInbox::Inbox::msg_by_mid; *modified = \&PublicInbox::Inbox::modified; -*recent = \&PublicInbox::Inbox::recent; *max_git_epoch = *nntp_usable = *msg_by_path = \&mm; # undef *isrch = *search = \&mm; # TODO *DESTROY = \&pause_dedupe; |