]> Sergey Matveev's repositories - public-inbox.git/commit
treewide: ditch inbox->recent method
authorEric Wong <e@80x24.org>
Mon, 29 Aug 2022 09:26:38 +0000 (09:26 +0000)
committerEric Wong <e@80x24.org>
Mon, 29 Aug 2022 19:05:49 +0000 (19:05 +0000)
commitbe55542acc90572da54d828313c2416ae1c902b1
tree001d9a6f38a16a023bd6f6b7a32a30403442a1ca
parenta66a2eddda8ffbb7679f4ef80dba7c7d377cda84
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.
lib/PublicInbox/ExtSearch.pm
lib/PublicInbox/Inbox.pm
lib/PublicInbox/LeiSavedSearch.pm
lib/PublicInbox/LeiXSearch.pm
lib/PublicInbox/View.pm
t/convert-compact.t
t/indexlevels-mirror.t
t/lei_xsearch.t
t/replace.t
t/v1-add-remove-add.t
t/v2-add-remove-add.t