lib/PublicInbox/InboxWritable.pm | 4 ++++ lib/PublicInbox/Xapcmd.pm | 2 +- script/public-inbox-edit | 4 ++-- diff --git a/lib/PublicInbox/InboxWritable.pm b/lib/PublicInbox/InboxWritable.pm index c73910acb30a754329bf5ce58e998a8775642b3f..d83912511451af8f5b14fc295616c88faa43ef1c 100644 --- a/lib/PublicInbox/InboxWritable.pm +++ b/lib/PublicInbox/InboxWritable.pm @@ -257,4 +257,8 @@ my $umask = _umask_for($perm); $self->{umask} = $umask; } +sub cleanup ($) { + delete @{$_[0]}{qw(over mm git search)}; +} + 1; diff --git a/lib/PublicInbox/Xapcmd.pm b/lib/PublicInbox/Xapcmd.pm index c807bf1014aef78d63f814ed17b4d7ead2840b00..77f0524e2b9a0fc42d67e935bc9309c735489d8f 100644 --- a/lib/PublicInbox/Xapcmd.pm +++ b/lib/PublicInbox/Xapcmd.pm @@ -234,7 +234,7 @@ prepare_reindex($ibx, $im, $reindex); $im->lock_release; } - delete($ibx->{$_}) for (qw(mm over search)); # cleanup + $ibx->cleanup; process_queue(\@q, $cb, $max, $opt); $im->lock_acquire if !$opt->{-coarse_lock}; commit_changes($ibx, $im, $tmp, $opt); diff --git a/script/public-inbox-edit b/script/public-inbox-edit index b437b3c0ac1f23c3408bb33e384b61623d5cea7c..1900b267fbaed598867dfd66917bad778a0c207f 100755 --- a/script/public-inbox-edit +++ b/script/public-inbox-edit @@ -57,7 +57,7 @@ my $cid = content_id($mime); my $tuple = [ $ibx, $smsg ]; push @{$found->{$cid} ||= []}, $tuple } - delete @$ibx{qw(over mm git search)}; # cleanup + PublicInbox::InboxWritable::cleanup($ibx); } $found; } @@ -131,7 +131,7 @@ $edit_fh->autoflush(1); my $edit_fn = $edit_fh->filename; my ($ibx, $smsg) = @{$to_edit->[0]}; my $old_raw = $ibx->msg_by_smsg($smsg); - delete @$ibx{qw(over mm git search)}; # cleanup + PublicInbox::InboxWritable::cleanup($ibx); my $tmp = $$old_raw; if (!$opt->{raw}) {