From: Jonathan Nieder <jrnieder@gmail.com>
Date: Mon, 29 Nov 2010 16:44:35 GMT
Subject: Re: [PATCH 3/6] web--browse: split valid_tool list
Message-ID: <20101129164435.GI8037@burratino>
In-Reply-To: <1291042075-19983-4-git-send-email-giuseppe.bilotta@gmail.com>

Giuseppe Bilotta wrote:

> --- a/git-web--browse.sh
> +++ b/git-web--browse.sh
> @@ -31,7 +31,8 @@ valid_custom_tool()
>  
>  valid_tool() {
>  	case "$1" in
> -	firefox | iceweasel | chrome | google-chrome | chromium | konqueror | w3m | links | lynx | dillo | open | start)
> +	firefox|iceweasel|chrome|google-chrome|chromium\
> +	|konqueror|w3m|links|lynx|dillo|open|start)

micronit: I think this looks better as

	firefox|iceweasel|chrome|google-chrome|chromium| \
	konqueror|w3m|links|lynx|dillo|open|start)

, with the | on the end of the first line.