lei_mail_sync: rely on flock(2), avoid IPC

Since 44917fdd24a8bec1 ("lei_mail_sync: do not use transactions"),
relying on lei/store to serialize access was a pointless endeavor.

Rely on flock(2) to serialize multiple writers since (in my
experience) it's the easiest way to deal with parallel writers
when using SQLite.  This allows us to simplify existing callers
while speeding up 'lei refresh-mail-sync --all=local' by 5% or
so.

1 files changed, 2 insertions, 4 deletions

diff --git a/lib/PublicInbox/LeiForgetMailSync.pm b/lib/PublicInbox/LeiForgetMailSync.pm
index 2b4e58a9..701f48d2 100644
--- a/lib/PublicInbox/LeiForgetMailSync.pm
+++ b/lib/PublicInbox/LeiForgetMailSync.pm
@@ -15,13 +15,11 @@ use PublicInbox::LeiExportKw;
 sub lei_forget_mail_sync {
         my ($lei, @folders) = @_;
         my $lms = $lei->lms or return;
-        my $sto = $lei->_lei_store or return; # may disappear due to race
-        $sto->write_prepare($lei);
+        $lms->lms_write_prepare;
         my $err = $lms->arg2folder($lei, \@folders);
         $lei->qerr(@{$err->{qerr}}) if $err->{qerr};
         return $lei->fail($err->{fail}) if $err->{fail};
-        $sto->ipc_do('lms_forget_folders', @folders);
-        $lei->sto_done_request;
+        $lms->forget_folders(@folders);
 }
 
 *_complete_forget_mail_sync = \&PublicInbox::LeiExportKw::_complete_export_kw;