]> Sergey Matveev's repositories - public-inbox.git/commitdiff
t/edit.t: drop redundant "delete local $ENV{...}"
authorEric Wong <e@80x24.org>
Mon, 16 Dec 2019 08:07:34 +0000 (08:07 +0000)
committerEric Wong <e@80x24.org>
Tue, 17 Dec 2019 02:53:06 +0000 (02:53 +0000)
"delete local" is only in Perl v5.11.0, and we only depend on
Perl v5.10.1.  We already localize and delete it as two separate
statements immediately above.

I wish this was hidden behind a "use feature" flag like other
new-fangled things :<.  Oh well, I think the oldest Perl actually
in use for this project is 5.16 (CentOS 7.x).

t/edit.t

index 02df6cda68e3d75f3899becc227b62c1935a4d11..122aa19f0b745d6d1dcd50632b57f718300553b8 100644 (file)
--- a/t/edit.t
+++ b/t/edit.t
@@ -129,7 +129,6 @@ $t = 'mailEditor set in config'; {
        is($rc, 0, 'set publicinbox.mailEditor');
        local $ENV{MAIL_EDITOR};
        delete $ENV{MAIL_EDITOR};
-       delete local $ENV{MAIL_EDITOR};
        local $ENV{GIT_EDITOR} = 'echo should not run';
        $cmd = [ '-edit', "-m$mid", $inboxdir ];
        ok(run_script($cmd, undef, $opt), "$t edited message");