On Fri, Jun 26, 2015 at 3:27 PM, Junio C Hamano <gits...@pobox.com> wrote:
> Stefan Beller <sbel...@google.com> writes:
>
>> Signed-off-by: Stefan Beller <sbel...@google.com>
>
> Thanks.  How did you find this (does the auto &&-chain test apply to
> t/perf stuff as well)?

Apparently the &&-chain tests for it as I got a warning for it while
benchmarking some changes in ALLOC_GROW. (which originally
should have fixed the coverity false positives, but I was side tracked
wondering about performance)

>
> Will queue.
>
>> ---
>>  t/perf/p5310-pack-bitmaps.sh | 6 +++---
>>  1 file changed, 3 insertions(+), 3 deletions(-)
>>
>> diff --git a/t/perf/p5310-pack-bitmaps.sh b/t/perf/p5310-pack-bitmaps.sh
>> index f8ed857..de2a224 100755
>> --- a/t/perf/p5310-pack-bitmaps.sh
>> +++ b/t/perf/p5310-pack-bitmaps.sh
>> @@ -39,14 +39,14 @@ test_expect_success 'create partial bitmap state' '
>>
>>       # now kill off all of the refs and pretend we had
>>       # just the one tip
>> -     rm -rf .git/logs .git/refs/* .git/packed-refs
>> -     git update-ref HEAD $cutoff
>> +     rm -rf .git/logs .git/refs/* .git/packed-refs &&
>> +     git update-ref HEAD $cutoff &&
>>
>>       # and then repack, which will leave us with a nice
>>       # big bitmap pack of the "old" history, and all of
>>       # the new history will be loose, as if it had been pushed
>>       # up incrementally and exploded via unpack-objects
>> -     git repack -Ad
>> +     git repack -Ad &&
>>
>>       # and now restore our original tip, as if the pushes
>>       # had happened
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to