inbox: name variable for values loop iterator

->on_inbox_unlock callbacks could clobber $_, and this seems to
fix a problem with -extindex --watch failing to index some
inboxes after SIGHUP reload.

(cherry picked from commit b5e960f50289434025f5904c8c1311e4c8a02b82)

diff --git a/lib/PublicInbox/Inbox.pm b/lib/PublicInbox/Inbox.pm
index b0894a7d6414d038d820afa4533a2c2f69321ddd..e9efd29d40a4b46f993bef2075653f632cb43544 100644 (file)
--- a/lib/PublicInbox/Inbox.pm
+++ b/lib/PublicInbox/Inbox.pm
@@ -422,8 +422,8 @@ sub on_unlock {
        my ($self) = @_;
        check_inodes($self);
        my $subs = $self->{unlock_subs} or return;
        my ($self) = @_;
        check_inodes($self);
        my $subs = $self->{unlock_subs} or return;

-       for (values %$subs) {
-               eval { $_->on_inbox_unlock($self) };
+       for my $obj (values %$subs) {
+               eval { $obj->on_inbox_unlock($self) };

                warn "E: $@ ($self->{inboxdir})\n" if $@;
        }
 }
                warn "E: $@ ($self->{inboxdir})\n" if $@;
        }
 }