about summary refs log tree commit homepage
path: root/lib/PublicInbox/LeiConvert.pm
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2021-03-31 07:45:51 +0000
committerEric Wong <e@80x24.org>2021-03-31 22:27:19 +0000
commit8fa51e510fbe629a05c2da82482053f77ece7de5 (patch)
treec25e83a48644e0665595a80343b352c083f84159 /lib/PublicInbox/LeiConvert.pm
parentb9dfb42d9f504c36693377f6beea98e954e7c5a9 (diff)
downloadpublic-inbox-8fa51e510fbe629a05c2da82482053f77ece7de5.tar.gz
No point in sending a command for every input when a
single one will do.  We'll also trigger LeiStore->done
sooner in the worker rather than later.
Diffstat (limited to 'lib/PublicInbox/LeiConvert.pm')
-rw-r--r--lib/PublicInbox/LeiConvert.pm11
1 files changed, 5 insertions, 6 deletions
diff --git a/lib/PublicInbox/LeiConvert.pm b/lib/PublicInbox/LeiConvert.pm
index fb7a2f3b..0ce49ea9 100644
--- a/lib/PublicInbox/LeiConvert.pm
+++ b/lib/PublicInbox/LeiConvert.pm
@@ -7,6 +7,7 @@ use strict;
 use v5.10.1;
 use parent qw(PublicInbox::IPC PublicInbox::LeiInput);
 use PublicInbox::LeiOverview;
+use PublicInbox::DS;
 
 # /^input_/ subs are used by PublicInbox::LeiInput
 
@@ -32,12 +33,10 @@ sub input_maildir_cb {
         $self->{wcb}->(undef, { kw => $kw }, $eml);
 }
 
-sub do_convert { # via wq_do
+sub process_inputs { # via wq_do
         my ($self) = @_;
-        $PublicInbox::DS::in_loop = 0; # force synchronous dwaitpid
-        for my $input (@{$self->{inputs}}) {
-                $self->input_path_url($input);
-        }
+        local $PublicInbox::DS::in_loop = 0; # force synchronous dwaitpid
+        $self->SUPER::process_inputs;
         delete $self->{lei}->{1};
         delete $self->{wcb}; # commit
 }
@@ -55,7 +54,7 @@ sub lei_convert { # the main "lei convert" method
         $self->prepare_inputs($lei, \@inputs) or return;
         my ($op_c, $ops) = $lei->workers_start($self, 'lei_convert', 1);
         $lei->{cnv} = $self;
-        $self->wq_io_do('do_convert', []);
+        $self->wq_io_do('process_inputs', []);
         $self->wq_close(1);
         $op_c->op_wait_event($ops);
 }