git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: Re* [PATCH v4] userdiff: improve java hunk header regex

From
Junio C Hamano <gitster@pobox.com>
Date
Aug 11, 2021, 16:04 UTC
Message-ID
<xmqqv94c9ddm.fsf@gitster.g>
In-Reply-To
<157a0c35-1c82-9a2e-3bcd-ae6059ec71bd@kdbg.org>
Johannes Sixt <j6t@kdbg.org> writes:
Show 8 quoted lines
>> + * When writing or updating patterns, assume that the contents these
>> + * patterns are applied to are syntactically correct.  You do not have
>> + * to implement all syntactical corner cases---the patterns have to be
>> + * sufficiently permissive.
>> + */
>
> IMO, as written, the comment falls short of suggesting that patterns can
> be simple. How about appending "and can be simple"?
    The patterns can be simple without implementing all syntactical
    corner cases, as long as they are sufficiently permissive.
perhaps?
Thanks.
Previous: Johannes SixtNext: Johannes Sixt
Message 5 of 9 in “userdiff: improve java hunk header regex”
  1. userdiff: improve java hunk header regexTassilo Horn, Aug 10, 2021
  2. Johannes SixtAug 10, 2021
  3. Re* [PATCH v4] userdiff: improve java hunk header regexJunio C Hamano, Aug 10, 2021
  4. Johannes SixtAug 11, 2021
  5. Junio C HamanoAug 11, 2021
  6. Johannes SixtAug 11, 2021
  7. Tassilo HornAug 11, 2021
  8. Johannes SixtAug 11, 2021
  9. Tassilo HornAug 11, 2021

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.