Re: [PATCH 5/5] test-run-command: Increase test coverage

2015-10-20 Thread Stefan Beller
On Tue, Oct 20, 2015 at 11:53 AM, Junio C Hamano wrote: > > proc.argv = (const char **)argv + 3; > > or > > proc.argv = (const char **)&argv[3]; ok, will fix > > Given the line immediately before refers to argv[2], the latter > might be easier on the eyes to follow. > > In what w

Re: [PATCH 5/5] test-run-command: Increase test coverage

2015-10-20 Thread Junio C Hamano
Stefan Beller writes: > Signed-off-by: Stefan Beller > --- > t/t0061-run-command.sh | 16 +--- > test-run-command.c | 12 > 2 files changed, 21 insertions(+), 7 deletions(-) > > diff --git a/t/t0061-run-command.sh b/t/t0061-run-command.sh > index 0af77cd..f27ada7 10

[PATCH 5/5] test-run-command: Increase test coverage

2015-10-19 Thread Stefan Beller
Signed-off-by: Stefan Beller --- t/t0061-run-command.sh | 16 +--- test-run-command.c | 12 2 files changed, 21 insertions(+), 7 deletions(-) diff --git a/t/t0061-run-command.sh b/t/t0061-run-command.sh index 0af77cd..f27ada7 100755 --- a/t/t0061-run-command.sh +++ b