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

From: Stephen Boyd <[email-masked]>
Date: Mon, 13 Apr 2009 16:39:52 -0700

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?