From: Junio C Hamano Date: Sun, 27 Jan 2013 03:15:17 GMT Subject: Re: [PATCH v3 2/2] mergetools: Simplify how we handle "vim" and "defaults" Message-ID: <7vobgbz58a.fsf@alter.siamese.dyndns.org> In-Reply-To: <1359247573-75825-1-git-send-email-davvid@gmail.com> David Aguilar writes: > @@ -44,19 +46,9 @@ valid_tool () { > } > > setup_tool () { > - case "$1" in > - vim*|gvim*) > - tool=vim > - ;; > - *) > - tool="$1" > - ;; > - esac This part was an eyesore every time I looked at mergetools scripts. Good riddance. Is there still other special case like this, or was this the last one? Thanks, both of you, for working on this.