From: Junio C Hamano <gitster@pobox.com>
Date: Fri, 21 Sep 2018 15:59:12 GMT
Subject: Re: [RFC PATCH v4 1/3] Add support for nested aliases
Message-ID: <xmqqa7oayh7z.fsf@gitster-ct.c.googlers.com>
In-Reply-To: <9f7aa1f1-2493-42f3-19b7-cfa2efadbb14@gmx.de>

Tim Schumacher <timschumi@gmx.de> writes:

> it is located at the top of the while() loop. Giving an example is nice, but wouldn't
> it be better to say something like the following?
>
> 	/*
> 	 * Check if av[0] is a command before seeing if it is an
> 	 * alias to avoid taking over existing commands
> 	 */

If we have more concrete and constructive things to explain why we
choose to forbid it, that may be worth saying, but I agree that it
does not add much value to this comment to declare that an attempt
to take over existing commands is "insane".

Thanks.