convert: avoid redundant "done\n" statement for fast-import

This bug was hidden due to timing problems with eatmydata or
running with tmpfs for TMPDIR.

diff --git a/script/public-inbox-convert b/script/public-inbox-convert
index 2b0a385e15cf70615f73403497a45819658e1b35..e6fb4f5f3cb11868e9ad72092bfc5bcce7faa0ec 100755 (executable)
--- a/script/public-inbox-convert
+++ b/script/public-inbox-convert
@@ -91,9 +91,8 @@ while (<$rd>) {
                        $from = $1;
                        # no next
                }
-       } elsif ($_ eq "done\n") {
-               last;
        }
+       last if $_ eq "done\n";
        $w->print($_) or $im->wfail;
 }
 $w = $r = undef;