(off-list ancestor, not in this archive)
Re: t3404 static check of bad SHA-1 failure · Junio C Hamano <hidden> · 2016-06-16
[PATCH 0/6] test -z/-n quoting fix + misc cleanups · Jeff King <hidden> · 2016-06-16
[PATCH 2/6] t9100,t3419: enclose all test code in single-quotes · Jeff King <hidden> · 2016-06-16
[PATCH 3/6] t9107: use "return 1" instead of "exit 1" · Jeff King <hidden> · 2016-06-16
Re: [PATCH 3/6] t9107: use "return 1" instead of "exit 1" · Eric Wong <hidden> · 2016-06-16
Re: [PATCH 3/6] t9107: use "return 1" instead of "exit 1" · Eric Sunshine <hidden> · 2016-06-16
Re: [PATCH 3/6] t9107: use "return 1" instead of "exit 1" · Jeff King <hidden> · 2016-06-16
[PATCH 1/6] t/lib-git-svn: drop $remote_git_svn and $git_svn_id · Jeff King <hidden> · 2016-06-16
[PATCH 5/6] t9103: modernize test style · Jeff King <hidden> · 2016-06-16
[PATCH 4/6] t9107: switch inverted single/double quotes in test · Jeff King <hidden> · 2016-06-16
[PATCH 6/6] always quote shell arguments to test -z/-n · Jeff King <hidden> · 2016-06-16

Re: [PATCH 3/6] t9107: use "return 1" instead of "exit 1"

From: Eric Wong <hidden>
Date: 2016-06-16 02:19:23

Possibly related (same subject, not in this thread)

2016-06-16 · Re: [PATCH 3/6] t9107: use "return 1" instead of "exit 1" · Junio C Hamano <hidden>
2016-06-16 · Re: [PATCH 3/6] t9107: use "return 1" instead of "exit 1" · Jeff King <hidden>

Jeff King [off-list ref] wrote:

+			git diff --exit-code refs/remotes/origin/$i refs/remotes/origin/$j ||
+				return 1

80 columns; but I guess Junio picked it up, already.
Otherwise the rest of the series looks good.  Thanks.