From: Barret Rhoden <brho@google.com>
Date: Mon, 01 Jul 2019 14:16:06 GMT
Subject: Re: [PATCH] t8014: remove unnecessary braces
Message-ID: <0835ac2f-59aa-a9bb-5e47-6617dae1f810@google.com>
In-Reply-To: <20190630181732.4128-1-michael@platin.gs>

I'll squash this fix in for v10.

On 6/30/19 2:17 PM, michael@platin.gs wrote:
> From: Michael Platings <michael@platin.gs>
> 
> Signed-off-by: Michael Platings <michael@platin.gs>
> ---
>   t/t8014-blame-ignore-fuzzy.sh | 12 ++++++------
>   1 file changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/t/t8014-blame-ignore-fuzzy.sh b/t/t8014-blame-ignore-fuzzy.sh
> index 6f1a94caef..6e61882b6f 100755
> --- a/t/t8014-blame-ignore-fuzzy.sh
> +++ b/t/t8014-blame-ignore-fuzzy.sh
> @@ -298,28 +298,28 @@ EOF
>   last_test=13
>   
>   test_expect_success setup '
> -	{ for i in $(test_seq 2 $last_test)
> +	for i in $(test_seq 2 $last_test)
>   	do
>   		# Append each line in a separate commit to make it easy to
>   		# check which original line the blame output relates to.
>   
>   		line_count=0 &&
> -		{ while IFS= read line
> +		while IFS= read line
>   		do
>   			line_count=$((line_count+1)) &&
>   			echo "$line" >>"$i" &&
>   			git add "$i" &&
>   			test_tick &&
>   			GIT_AUTHOR_NAME="$line_count" git commit -m "$line_count"
> -		done } <"a$i"
> -	done } &&
> +		done <"a$i"
> +	done &&
>   
> -	{ for i in $(test_seq 2 $last_test)
> +	for i in $(test_seq 2 $last_test)
>   	do
>   		# Overwrite the files with the final content.
>   		cp b$i $i &&
>   		git add $i
> -	done } &&
> +	done &&
>   	test_tick &&
>   
>   	# Commit the final content all at once so it can all be
>