diff options
author | Eric Wong <e@80x24.org> | 2021-03-31 07:45:51 +0000 |
---|---|---|
committer | Eric Wong <e@80x24.org> | 2021-03-31 22:27:19 +0000 |
commit | 8fa51e510fbe629a05c2da82482053f77ece7de5 (patch) | |
tree | c25e83a48644e0665595a80343b352c083f84159 /lib/PublicInbox/LeiConvert.pm | |
parent | b9dfb42d9f504c36693377f6beea98e954e7c5a9 (diff) | |
download | public-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.pm | 11 |
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); } |