Re: [PATCH v3 2/2] mergetools: Simplify how we handle "vim" and "defaults"

From: Junio C Hamano <gitster@pobox.com>
Date: Jan 27, 2013, 03:15 UTC
Message-ID: <7vobgbz58a.fsf@alter.siamese.dyndns.org>
In-Reply-To: <1359247573-75825-1-git-send-email-davvid@gmail.com>

David Aguilar <davvid@gmail.com> writes:

Show 12 quoted lines

> @@ -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.