]> git.ipfire.org Git - thirdparty/git.git/commitdiff
t5310: fix "; do" style
authorJeff King <peff@peff.net>
Tue, 9 May 2017 02:59:46 +0000 (22:59 -0400)
committerJunio C Hamano <gitster@pobox.com>
Tue, 9 May 2017 03:09:03 +0000 (12:09 +0900)
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 <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
t/t5310-pack-bitmaps.sh

index c3ddfa217fc006b4ba4feae6859dcf5b8b81674e..20e2473a03b645d690b25598bc0cb2e421034b6c 100755 (executable)
@@ -20,11 +20,13 @@ has_any () {
 }
 
 test_expect_success 'setup repo with moderate-sized history' '
-       for i in $(test_seq 1 10); do
+       for i in $(test_seq 1 10)
+       do
                test_commit $i
        done &&
        git checkout -b other HEAD~5 &&
-       for i in $(test_seq 1 10); do
+       for i in $(test_seq 1 10)
+       do
                test_commit side-$i
        done &&
        git checkout master &&
@@ -104,7 +106,8 @@ test_expect_success 'clone from bitmapped repository' '
 '
 
 test_expect_success 'setup further non-bitmapped commits' '
-       for i in $(test_seq 1 10); do
+       for i in $(test_seq 1 10)
+       do
                test_commit further-$i
        done
 '
@@ -300,7 +303,8 @@ test_expect_success 'set up reusable pack' '
 
 test_expect_success 'pack reuse respects --honor-pack-keep' '
        test_when_finished "rm -f .git/objects/pack/*.keep" &&
-       for i in .git/objects/pack/*.pack; do
+       for i in .git/objects/pack/*.pack
+       do
                >${i%.pack}.keep
        done &&
        reusable_pack --honor-pack-keep >empty.pack &&