From b40f4f1f81f79577e19e9926e14979625303eb59 Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Tue, 30 Mar 2021 09:10:55 +0000 Subject: lei q: avoid redundant default setting for sort with l2m No point in munging user-supplied $lei->{opt} when %mset_opt exists. We'll be depending on docid being in descending order for saved search support. --- lib/PublicInbox/LeiQuery.pm | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'lib/PublicInbox/LeiQuery.pm') diff --git a/lib/PublicInbox/LeiQuery.pm b/lib/PublicInbox/LeiQuery.pm index 5376c7f8..3a437bf0 100644 --- a/lib/PublicInbox/LeiQuery.pm +++ b/lib/PublicInbox/LeiQuery.pm @@ -128,8 +128,8 @@ sub lei_q { die "unrecognized --sort=$sort\n"; } } - # descending docid order - $mset_opt{relevance} //= -2 if $opt->{threads}; + # descending docid order is cheapest, MUA controls sorting order + $mset_opt{relevance} //= -2 if $self->{l2m} || $opt->{threads}; $self->{mset_opt} = \%mset_opt; if ($opt->{stdin}) { -- cgit v1.2.3-24-ge0c7