Subject: Re: [PATCH] git-submodule.sh: avoid "test <cond> -a/-o <cond>"

From: Torsten Bögershausen <[email-masked]>
Date: Tue, 10 Jun 2014 16:52:46 +0200

On 2014-06-10 14.28, Elia Pinto wrote:
[]
>  		# before the first commit: compare with an empty tree
>  		head=$(git hash-object -w -t tree --stdin </dev/null)
> @@ -1056,13 +1056,17 @@ cmd_summary() {
>  		while read mod_src mod_dst sha1_src sha1_dst status sm_path
>  		do
>  			# Always show modules deleted or type-changed (blob<->module)
> -			test $status = D -o $status = T && echo "$sm_path" && continue
> +			case "$status" in
> +			([DT])
Does this look strange? ^
Should it be
case "$status" in
D|T)