On Tue, 8 Mar 2022 17:19:14 GMT, Ian Graves <igra...@openjdk.org> wrote:
>> Proposed change in behavior to correct inconsistencies between `\w` and `\b` >> metacharacters > > Ian Graves has updated the pull request with a new target base due to a merge > or a rebase. The pull request now contains six commits: > > - Merge branch 'master' of https://github.com/openjdk/jdk into bug-8264160 > - Bug id's to header and some other cleanup > - Fixing bad javadoc > - Merge remote-tracking branch 'upstream/master' into bug-8264160 > - Updating spec > - Proposed change for \b metacharacter The changes to Pattern seem OK pending the final CSR approval. Looks like your CSR is not quite ready for review as their is no info in the compatibility section. test/jdk/java/util/regex/RegExTest.java line 65: > 63: import java.util.stream.Stream; > 64: > 65: import org.testng.annotations.DataProvider; I don't see a DataProvider being used anywhere, perhaps you were and then decided against it? test/jdk/java/util/regex/RegExTest.java line 4556: > 4554: } > 4555: > 4556: //This test is for 8264160 Even if you are not going to do a pass through existing tests to provide a comment of the tests intent as part of this PR, please do so for your additional tests. Future maintainers will thank you :-) ------------- PR: https://git.openjdk.java.net/jdk/pull/7539
