Merge branch 'lc/shell-default-value-noexpand'

Fix unnecessarily waste in the idiomatic use of ': ${VAR=default}'
to set the default value, without enclosing it in double quotes.

* lc/shell-default-value-noexpand:
sh-setup: enclose setting of ${VAR=default} in double-quotes

diff --combined git-sh-setup.sh
index c48139a49420ea2318e23d8667eb20bae9772463,c918ed779b8436976ee6c0a2a7489fc5d4ecc20f..85db5f1c63c3c6fa0fff95a747538e47f02ec46d
--- 1/git-sh-setup.sh
--- 2/git-sh-setup.sh
+++ b/git-sh-setup.sh
@@@ -160,19 -160,19 +160,19 @@@ git_pager() 
        else
                GIT_PAGER=cat
        fi
-       : ${LESS=-FRX}
-       : ${LV=-c}
+       : "${LESS=-FRX}"
+       : "${LV=-c}"
        export LESS LV
  
        eval "$GIT_PAGER" '"$@"'
  }
  
  sane_grep () {
 -      GREP_OPTIONS= LC_ALL=C grep "$@"
 +      GREP_OPTIONS= LC_ALL=C grep @@SANE_TEXT_GREP@@ "$@"
  }
  
  sane_egrep () {
 -      GREP_OPTIONS= LC_ALL=C egrep "$@"
 +      GREP_OPTIONS= LC_ALL=C egrep @@SANE_TEXT_GREP@@ "$@"
  }
  
  is_bare_repository () {
@@@ -344,7 -344,7 +344,7 @@@ git_dir_init () 
                echo >&2 "Unable to determine absolute path of git directory"
                exit 1
        }
-       : ${GIT_OBJECT_DIRECTORY="$(git rev-parse --git-path objects)"}
+       : "${GIT_OBJECT_DIRECTORY="$(git rev-parse --git-path objects)"}"
  }
  
  if test -z "$NONGIT_OK"