Re: [PATCH RFC3 08/13] send-email: Remove superfluous `my $editor = ...'

Michael Witten wrote:

-	my $editor = $ENV{GIT_EDITOR} || Git::config(@repo, "core.editor") || $ENV{VISUAL} || $ENV{EDITOR} || "vi";

I don't want to start an editor war, but why is 'vi' here? It seems that one of the previous four should be set at all times, correct?