Skip to content

Commit da5a1f8

Browse files
peffgitster
authored andcommitted
t5310: fix "; do" style
Our usual shell style is to put the "do" of a loop on its own line, like: while $cond do something done instead of: while $cond; do something done We have a bit of both in our code base, but the former is what's in CodingGuidelines (and outnumbers the latter in t/ by about 6:1). Signed-off-by: Jeff King <[email protected]> Signed-off-by: Junio C Hamano <[email protected]>
1 parent 9df4a60 commit da5a1f8

File tree

1 file changed

+8
-4
lines changed

1 file changed

+8
-4
lines changed

t/t5310-pack-bitmaps.sh

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -20,11 +20,13 @@ has_any () {
2020
}
2121

2222
test_expect_success 'setup repo with moderate-sized history' '
23-
for i in $(test_seq 1 10); do
23+
for i in $(test_seq 1 10)
24+
do
2425
test_commit $i
2526
done &&
2627
git checkout -b other HEAD~5 &&
27-
for i in $(test_seq 1 10); do
28+
for i in $(test_seq 1 10)
29+
do
2830
test_commit side-$i
2931
done &&
3032
git checkout master &&
@@ -104,7 +106,8 @@ test_expect_success 'clone from bitmapped repository' '
104106
'
105107

106108
test_expect_success 'setup further non-bitmapped commits' '
107-
for i in $(test_seq 1 10); do
109+
for i in $(test_seq 1 10)
110+
do
108111
test_commit further-$i
109112
done
110113
'
@@ -300,7 +303,8 @@ test_expect_success 'set up reusable pack' '
300303

301304
test_expect_success 'pack reuse respects --honor-pack-keep' '
302305
test_when_finished "rm -f .git/objects/pack/*.keep" &&
303-
for i in .git/objects/pack/*.pack; do
306+
for i in .git/objects/pack/*.pack
307+
do
304308
>${i%.pack}.keep
305309
done &&
306310
reusable_pack --honor-pack-keep >empty.pack &&

0 commit comments

Comments
 (0)