Jakub Narebski <jnareb@xxxxxxxxx> writes: > if [ -n "$GIT_SSL_NO_VERIFY" ]; then > - curl_extra_args="-k" > + curl_extra_args="-k" > fi Things like this makes some sense... > if test -n "$use_separate_remote" && > - branch_name=`expr "z$name" : 'zheads/\(.*\)'` > + branch_name=`expr "z$name" : 'zheads/\(.*\)'` > then ... but not this, which is not just indent but an alignment. > case "$2" in > '') > - usage ;; > + usage ;; > */*) > - echo >&2 "'$2' is not suitable for an origin name" > - exit 1 > + echo >&2 "'$2' is not suitable for an origin name" > + exit 1 Also shell scripts tend to become too deeply indented, and 4-column indentation helps to keep things within typical screen width. In short, not very enthused. - : send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html