about summary refs log tree commit homepage
path: root/lib/PublicInbox/LeiXSearch.pm
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2021-05-06 02:28:19 +0000
committerEric Wong <e@80x24.org>2021-05-06 16:56:44 +0000
commitde20dc26cd22271327e8a84818ff0774cecc1361 (patch)
tree19d11f43356a158b2e1db05234c638634de78d15 /lib/PublicInbox/LeiXSearch.pm
parent473b069c61b3d4936e2c5ed46fd7d348c9f6d1b7 (diff)
downloadpublic-inbox-de20dc26cd22271327e8a84818ff0774cecc1361.tar.gz
We must not accumulate mset totals for messages which
have already been counted.  Furthermore, the combined
search was being passed an extra arg and causing the
total to go missing.
Diffstat (limited to 'lib/PublicInbox/LeiXSearch.pm')
-rw-r--r--lib/PublicInbox/LeiXSearch.pm4
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/PublicInbox/LeiXSearch.pm b/lib/PublicInbox/LeiXSearch.pm
index 21b15025..e69a4edd 100644
--- a/lib/PublicInbox/LeiXSearch.pm
+++ b/lib/PublicInbox/LeiXSearch.pm
@@ -139,7 +139,7 @@ sub mset_progress {
                 pkt_do($lei->{pkt_op_p}, 'mset_progress', @_);
         } else { # single lei-daemon consumer
                 my ($desc, $mset_size, $mset_total_est) = @_;
-                $lei->{-mset_total} += $mset_size;
+                $lei->{-mset_total} += $mset_size if $mset_total_est ne '?';
                 $lei->qerr("# $desc $mset_size/$mset_total_est");
         }
 }
@@ -238,7 +238,7 @@ sub query_combined_mset { # non-parallel for non-"--threads" users
         do {
                 $mset = $self->mset($mo->{qstr}, $mo);
                 mset_progress($lei, 'xsearch', $mset->size,
-                                $mset->size, $mset->get_matches_estimated);
+                                $mset->get_matches_estimated);
                 wait_startq($lei); # wait for keyword updates
                 for my $mitem ($mset->items) {
                         my $smsg = smsg_for($self, $mitem) or next;