Re: [PATCH] Teach ignore machinery about pattern "/"

From: Junio C Hamano <gitster@pobox.com>
Date: May 27, 2012, 06:15 UTC
Message-ID: <7vwr3yp2t8.fsf@alter.siamese.dyndns.org>
In-Reply-To: <CACsJy8BjUQsCVVdzZtJxA9Qu2G0izZa8dvP1EKsQQAHWXZbHdg@mail.gmail.com>

Nguyen Thai Ngoc Duy <pclouds@gmail.com> writes:

Show 13 quoted lines

>> When talking about an ambiguous expression X that can be in multiple ways
>> and each interpretation gives surprisingly different result, if there are
>> ways A, B and C to unambiguously spell each and every of its possible and
>> useful interpretations, and if one of the interpretations C is to ignore
>> the expression altogether, it is preferrable to either
>>
>> (1) warn if it does not trigger anything useful to write a no-op; or
>> (2) just ignore it if a no-op is a meaningful behaviour
>>
>> when you see X.
>
> except that we do the ignoring in (2) but I don't see this as a
> meaningful behavior.

That would mean you would want to update the code to do (1), not silently trigger a different behaviour.