Re: [PATCH] p5310: Fix broken && chain in performance test

2015-06-26 Thread Jeff King
On Fri, Jun 26, 2015 at 03:34:19PM -0700, Stefan Beller wrote: > > 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

Re: [PATCH] p5310: Fix broken && chain in performance test

2015-06-26 Thread Stefan Beller
On Fri, Jun 26, 2015 at 3:27 PM, Junio C Hamano wrote: > Stefan Beller writes: > >> Signed-off-by: Stefan Beller > > 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 so

Re: [PATCH] p5310: Fix broken && chain in performance test

2015-06-26 Thread Junio C Hamano
Stefan Beller writes: > Signed-off-by: Stefan Beller Thanks. How did you find this (does the auto &&-chain test apply to t/perf stuff as well)? Will queue. > --- > t/perf/p5310-pack-bitmaps.sh | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/t/perf/p5310-pack-b

[PATCH] p5310: Fix broken && chain in performance test

2015-06-26 Thread Stefan Beller
Signed-off-by: Stefan Beller --- 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 @