On Mon, 6 Jun 2022 15:41:06 GMT, Paul Sandoz <psan...@openjdk.org> wrote:
> Looks good. As a follow on PR I think it would be useful to add constants > `OFFSET_IN_RANGE` and `OFFSET_OUT_OF_RANGE`, then it becomes much clearer in > source and you can drop the `/* offsetInRange */` comment on the argument. Hi @PaulSandoz , thanks for the advice! I'v rebased the codes and added these two constants in the latest patch. Thanks again for the review! ------------- PR: https://git.openjdk.java.net/jdk/pull/8035