Re: [PATCH v2 18/19] t: add basic bitmap functionality tests

2013-10-30 Thread Jeff King
On Mon, Oct 28, 2013 at 11:13:47PM +0100, SZEDER Gábor wrote: + for i in `test_seq 1 10`; do + test_commit side-$i + done Sorry, style nitpick: could you rewrite this command substitution using $() like a few lines above? That is a nitpick. :P But sure, I've squashed it

Re: [PATCH v2 18/19] t: add basic bitmap functionality tests

2013-10-28 Thread SZEDER Gábor
On Fri, Oct 25, 2013 at 02:04:38AM -0400, Jeff King wrote: diff --git a/t/t5310-pack-bitmaps.sh b/t/t5310-pack-bitmaps.sh new file mode 100755 index 000..0868725 --- /dev/null +++ b/t/t5310-pack-bitmaps.sh @@ -0,0 +1,114 @@ +#!/bin/sh + +test_description='exercise basic bitmap

[PATCH v2 18/19] t: add basic bitmap functionality tests

2013-10-25 Thread Jeff King
Now that we can read and write bitmaps, we can exercise them with some basic functionality tests. These tests aren't particularly useful for seeing the benefit, as the test repo is too small for it to make a difference. However, we can at least check that using bitmaps does not break anything.