Code

web--browse: coding style
authorGiuseppe Bilotta <giuseppe.bilotta@gmail.com>
Fri, 3 Dec 2010 16:47:36 +0000 (17:47 +0100)
committerJunio C Hamano <gitster@pobox.com>
Fri, 3 Dec 2010 21:44:07 +0000 (13:44 -0800)
Retab and deindent choices in case statements.

Signed-off-by: Giuseppe Bilotta <giuseppe.bilotta@gmail.com>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
git-web--browse.sh

index 3fc4166b25714911b6b1294c7439da1e237e4918..7c4568fc236c1e76ff67a5ca0f55487052f844ae 100755 (executable)
@@ -31,11 +31,11 @@ valid_custom_tool()
 
 valid_tool() {
        case "$1" in
-               firefox | iceweasel | chrome | google-chrome | chromium | konqueror | w3m | links | lynx | dillo | open | start)
-                       ;; # happy
-               *)
-                       valid_custom_tool "$1" || return 1
-                       ;;
+       firefox | iceweasel | chrome | google-chrome | chromium | konqueror | w3m | links | lynx | dillo | open | start)
+               ;; # happy
+       *)
+               valid_custom_tool "$1" || return 1
+               ;;
        esac
 }
 
@@ -46,139 +46,139 @@ init_browser_path() {
 
 while test $# != 0
 do
-    case "$1" in
+       case "$1" in
        -b|--browser*|-t|--tool*)
-           case "$#,$1" in
+               case "$#,$1" in
                *,*=*)
-                   browser=`expr "z$1" : 'z-[^=]*=\(.*\)'`
-                   ;;
+                       browser=`expr "z$1" : 'z-[^=]*=\(.*\)'`
+                       ;;
                1,*)
-                   usage ;;
+                       usage ;;
                *)
-                   browser="$2"
-                   shift ;;
-           esac
-           ;;
+                       browser="$2"
+                       shift ;;
+               esac
+               ;;
        -c|--config*)
-           case "$#,$1" in
+               case "$#,$1" in
                *,*=*)
-                   conf=`expr "z$1" : 'z-[^=]*=\(.*\)'`
-                   ;;
+                       conf=`expr "z$1" : 'z-[^=]*=\(.*\)'`
+                       ;;
                1,*)
-                   usage ;;
+                       usage ;;
                *)
-                   conf="$2"
-                   shift ;;
-           esac
-           ;;
+                       conf="$2"
+                       shift ;;
+               esac
+               ;;
        --)
-           break
-           ;;
+               break
+               ;;
        -*)
-           usage
-           ;;
+               usage
+               ;;
        *)
-           break
-           ;;
-    esac
-    shift
+               break
+               ;;
+       esac
+       shift
 done
 
 test $# = 0 && usage
 
 if test -z "$browser"
 then
-    for opt in "$conf" "web.browser"
-    do
-       test -z "$opt" && continue
-       browser="`git config $opt`"
-       test -z "$browser" || break
-    done
-    if test -n "$browser" && ! valid_tool "$browser"; then
-       echo >&2 "git config option $opt set to unknown browser: $browser"
-       echo >&2 "Resetting to default..."
-       unset browser
-    fi
+       for opt in "$conf" "web.browser"
+       do
+               test -z "$opt" && continue
+               browser="`git config $opt`"
+               test -z "$browser" || break
+       done
+       if test -n "$browser" && ! valid_tool "$browser"; then
+               echo >&2 "git config option $opt set to unknown browser: $browser"
+               echo >&2 "Resetting to default..."
+               unset browser
+       fi
 fi
 
 if test -z "$browser" ; then
-    if test -n "$DISPLAY"; then
-       browser_candidates="firefox iceweasel google-chrome chrome chromium konqueror w3m links lynx dillo"
-       if test "$KDE_FULL_SESSION" = "true"; then
-           browser_candidates="konqueror $browser_candidates"
+       if test -n "$DISPLAY"; then
+               browser_candidates="firefox iceweasel google-chrome chrome chromium konqueror w3m links lynx dillo"
+               if test "$KDE_FULL_SESSION" = "true"; then
+                       browser_candidates="konqueror $browser_candidates"
+               fi
+       else
+               browser_candidates="w3m links lynx"
        fi
-    else
-       browser_candidates="w3m links lynx"
-    fi
-    # SECURITYSESSIONID indicates an OS X GUI login session
-    if test -n "$SECURITYSESSIONID" \
-           -o "$TERM_PROGRAM" = "Apple_Terminal" ; then
-       browser_candidates="open $browser_candidates"
-    fi
-    # /bin/start indicates MinGW
-    if test -x /bin/start; then
-       browser_candidates="start $browser_candidates"
-    fi
-
-    for i in $browser_candidates; do
-       init_browser_path $i
-       if type "$browser_path" > /dev/null 2>&1; then
-           browser=$i
-           break
+       # SECURITYSESSIONID indicates an OS X GUI login session
+       if test -n "$SECURITYSESSIONID" \
+               -o "$TERM_PROGRAM" = "Apple_Terminal" ; then
+               browser_candidates="open $browser_candidates"
        fi
-    done
-    test -z "$browser" && die "No known browser available."
+       # /bin/start indicates MinGW
+       if test -x /bin/start; then
+               browser_candidates="start $browser_candidates"
+       fi
+
+       for i in $browser_candidates; do
+               init_browser_path $i
+               if type "$browser_path" > /dev/null 2>&1; then
+                       browser=$i
+                       break
+               fi
+       done
+       test -z "$browser" && die "No known browser available."
 else
-    valid_tool "$browser" || die "Unknown browser '$browser'."
+       valid_tool "$browser" || die "Unknown browser '$browser'."
 
-    init_browser_path "$browser"
+       init_browser_path "$browser"
 
-    if test -z "$browser_cmd" && ! type "$browser_path" > /dev/null 2>&1; then
-       die "The browser $browser is not available as '$browser_path'."
-    fi
+       if test -z "$browser_cmd" && ! type "$browser_path" > /dev/null 2>&1; then
+               die "The browser $browser is not available as '$browser_path'."
+       fi
 fi
 
 case "$browser" in
-    firefox|iceweasel)
+firefox|iceweasel)
        # Check version because firefox < 2.0 does not support "-new-tab".
        vers=$(expr "$($browser_path -version)" : '.* \([0-9][0-9]*\)\..*')
        NEWTAB='-new-tab'
        test "$vers" -lt 2 && NEWTAB=''
        "$browser_path" $NEWTAB "$@" &
        ;;
-    google-chrome|chrome|chromium)
+google-chrome|chrome|chromium)
        # Actual command for chromium is chromium-browser.
        # No need to specify newTab. It's default in chromium
        eval "$browser_path" "$@" &
        ;;
-    konqueror)
+konqueror)
        case "$(basename "$browser_path")" in
-           konqueror)
+       konqueror)
                # It's simpler to use kfmclient to open a new tab in konqueror.
                browser_path="$(echo "$browser_path" | sed -e 's/konqueror$/kfmclient/')"
                type "$browser_path" > /dev/null 2>&1 || die "No '$browser_path' found."
                eval "$browser_path" newTab "$@"
                ;;
-           kfmclient)
+       kfmclient)
                eval "$browser_path" newTab "$@"
                ;;
-           *)
+       *)
                "$browser_path" "$@" &
                ;;
        esac
        ;;
-    w3m|links|lynx|open)
+w3m|links|lynx|open)
        eval "$browser_path" "$@"
        ;;
-    start)
-        exec "$browser_path" '"web-browse"' "$@"
-        ;;
-    dillo)
+start)
+       exec "$browser_path" '"web-browse"' "$@"
+       ;;
+dillo)
        "$browser_path" "$@" &
        ;;
-    *)
+*)
        if test -n "$browser_cmd"; then
-           ( eval $browser_cmd "$@" )
+               ( eval $browser_cmd "$@" )
        fi
        ;;
 esac