Re: RFR: 8333599: Improve description of \b matcher in j.u.r.Pattern

2024-06-10 Thread Raffaello Giulietti
On Fri, 7 Jun 2024 00:23:10 GMT, Stuart Marks wrote: >> A documentation-only change to match the original intent and the implemented >> behavior. > > Oh, this needs a CSR too, since it's a change to a normative assertion. > Should be pretty simple though. @stuart-marks A [CSR](https://bugs.ope

Re: RFR: 8333599: Improve description of \b matcher in j.u.r.Pattern

2024-06-06 Thread Alan Bateman
On Thu, 6 Jun 2024 18:16:14 GMT, Raffaello Giulietti wrote: > A documentation-only change to match the original intent and the implemented > behavior. Yes, this one needs a CSR. - Marked as reviewed by alanb (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/19583#pullreque

Re: RFR: 8333599: Improve description of \b matcher in j.u.r.Pattern

2024-06-06 Thread Stuart Marks
On Thu, 6 Jun 2024 18:16:14 GMT, Raffaello Giulietti wrote: > A documentation-only change to match the original intent and the implemented > behavior. Oh, this needs a CSR too, since it's a change to a normative assertion. Should be pretty simple though. - PR Comment: https://gi

Re: RFR: 8333599: Improve description of \b matcher in j.u.r.Pattern

2024-06-06 Thread Stuart Marks
On Thu, 6 Jun 2024 18:16:14 GMT, Raffaello Giulietti wrote: > A documentation-only change to match the original intent and the implemented > behavior. Looks good. The old version with the lookahead and lookbehind was hard to understand and it was inaccurate as well. The descriptive text is ea

RFR: 8333599: Improve description of \b matcher in j.u.r.Pattern

2024-06-06 Thread Raffaello Giulietti
A documentation-only change to match the original intent and the implemented behavior. - Commit messages: - 8333599: Improve description of \b matcher in j.u.r.Pattern Changes: https://git.openjdk.org/jdk/pull/19583/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=19583&ran