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

2 messages, 2 authors, 2021-08-11 · open the first message on its own page

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

From: Junio C Hamano <hidden>
Date: 2021-08-11 16:04:27

Johannes Sixt [off-list ref] writes:
quoted
+ * 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.

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

From: Johannes Sixt <hidden>
Date: 2021-08-11 20:32:29

Am 11.08.21 um 18:04 schrieb Junio C Hamano:
Johannes Sixt [off-list ref] writes:
quoted
quoted
+ * 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?
Perfect! Thank you.

-- Hannes
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help