Re: [PATCH] notes: remove trailing whitespace from editor template

From: Eric Sunshine <sunshine@sunshineco.com>
Date: May 27, 2025, 17:28 UTC
Message-ID: <CAPig+cScZ_0g7K-pQrxSseFX+fqjA6KAFf3nSi5cxbbCHhR5zQ@mail.gmail.com>
In-Reply-To: <xmqq5xhmvuol.fsf@gitster.g>

On Tue, May 27, 2025 at 1:19 PM Junio C Hamano <gitster@pobox.com> wrote:

Show 13 quoted lines

> Phillip Wood <phillip.wood123@gmail.com> writes:
> > Should that be " \$"? What you've got seems to work with dash but I'm
> > not sure if it is POSIX compliant or not.
>
> "2.6 Word Expansions" ends with this sentence:
>
>     If a '$' that is neither within single-quotes nor escaped by a
>     <backslash> is immediately followed by a <space>, <tab>, or a
>     <newline>, or is not followed by any character, the '$' shall be
>     treated as a literal character.
>
> Taken together with "2.2.3 Double-Quotes", I'd read it as blessing a
> lone '$' at the end of double-quoted string as a literal dollar sign.

Thanks for finding and posting this. I had read the POSIX shell language specification pretty closely when writing the lexer and parser for chainlint.pl and remembered the above statement, and had wanted to cite it after reading Phillip's initial reply but didn't manage to find it again in the limited time I had available for searching through the documentation.