inboxidle: remove needless check for {inboxdir}

->each_inbox will never attempt to iterate an object
without {inboxdir}, and simplify + short-circuit the
corresponding code

1 files changed, 3 insertions, 4 deletions

diff --git a/lib/PublicInbox/Config.pm b/lib/PublicInbox/Config.pm
index cafd9c3b..199ce019 100644
--- a/lib/PublicInbox/Config.pm
+++ b/lib/PublicInbox/Config.pm
@@ -391,9 +391,9 @@ EOF
                 }
         }
 
-        # backwards compatibility:
-        $ibx->{inboxdir} //= $self->{"$pfx.mainrepo"};
-        if (($ibx->{inboxdir} // '') =~ /\n/s) {
+        # "mainrepo" is backwards compatibility:
+        $ibx->{inboxdir} //= $self->{"$pfx.mainrepo"} // return;
+        if ($ibx->{inboxdir} =~ /\n/s) {
                 warn "E: `$ibx->{inboxdir}' must not contain `\\n'\n";
                 return;
         }
@@ -415,7 +415,6 @@ EOF
                 }
         }
 
-        return unless defined($ibx->{inboxdir});
         my $name = $pfx;
         $name =~ s/\Apublicinbox\.//;