[PATCH] t: don't redefine test_config() in various places

2013-03-20 Thread Ramkumar Ramachandra
test_config() is already a well-defined function in test-lib-functions.sh. Don't duplicate it unnecessarily in: t4018-diff-funcname.sh t7810-grep.sh t7811-grep-open.sh Signed-off-by: Ramkumar Ramachandra artag...@gmail.com --- Thanks, Junio. t/t4018-diff-funcname.sh | 5 -

Re: [PATCH] t: don't redefine test_config() in various places

2013-03-20 Thread Jonathan Nieder
Ramkumar Ramachandra wrote: Signed-off-by: Ramkumar Ramachandra artag...@gmail.com --- Thanks, Junio. t/t4018-diff-funcname.sh | 5 - t/t7810-grep.sh | 5 - t/t7811-grep-open.sh | 5 - 3 files changed, 15 deletions(-) Yeah, that looks like all of them. FWIW,

Re: [PATCH] t: don't redefine test_config() in various places

2013-03-20 Thread Junio C Hamano
Ramkumar Ramachandra artag...@gmail.com writes: test_config() is already a well-defined function in test-lib-functions.sh. Don't duplicate it unnecessarily in: t4018-diff-funcname.sh t7810-grep.sh t7811-grep-open.sh Signed-off-by: Ramkumar Ramachandra artag...@gmail.com ---

Re: [PATCH] t: don't redefine test_config() in various places

2013-03-20 Thread Ramkumar Ramachandra
Junio C Hamano wrote: Ramkumar Ramachandra artag...@gmail.com writes: test_config() is already a well-defined function in test-lib-functions.sh. Don't duplicate it unnecessarily in: t4018-diff-funcname.sh t7810-grep.sh t7811-grep-open.sh Signed-off-by: Ramkumar Ramachandra