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

Reply via email to