Subject: Re: [PATCH 2/4] Remove unecessary hard-coding of EDITOR=':' VISUAL=':' in some test suites.

From: Junio C Hamano <[email-masked]>
Date: Fri, 02 Nov 2007 13:09:56 -0700

Kristian Høgsberg <[email-masked]> writes:

> Signed-off-by: Kristian Høgsberg <[email-masked]>
> ---
>  t/t3501-revert-cherry-pick.sh |    4 ++--
>  t/t3901-i18n-patch.sh         |    8 ++++----
>  2 files changed, 6 insertions(+), 6 deletions(-)

This is a good patch, but a real commit message that says why
they are unnecessary is lacking.  Something like...

        Sourcing of ./test-lib.sh at the very beginning of test scripts
        already define and export them.