From 0514c910b0a33fe758cd92fd89fa79a3291bf5fc Mon Sep 17 00:00:00 2001 From: "Eric Wong (Contractor, The Linux Foundation)" Date: Mon, 19 Mar 2018 08:14:53 +0000 Subject: v2writable: ensure ->done is idempotent This matches Import::done behavior --- lib/PublicInbox/V2Writable.pm | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'lib') diff --git a/lib/PublicInbox/V2Writable.pm b/lib/PublicInbox/V2Writable.pm index 36901cde..5c104d8e 100644 --- a/lib/PublicInbox/V2Writable.pm +++ b/lib/PublicInbox/V2Writable.pm @@ -249,10 +249,11 @@ sub remove { sub done { my ($self) = @_; + my $locked = defined $self->{idx_parts}; my $im = delete $self->{im}; $im->done if $im; # PublicInbox::Import::done $self->searchidx_checkpoint(0); - $self->lock_release; + $self->lock_release if $locked; } sub checkpoint { -- cgit v1.2.3-24-ge0c7