On Thu, 20 Apr 2023 18:39:29 GMT, Naoto Sato <na...@openjdk.org> wrote:
> The current macro assumes the argument is `jint` because all locations pass > `jint`. However, this would not work if a wider type such as `jlong` is > passed. Removing the `unsigned` cast and make the condition explicit would > make the macro more robust. Looks fine. ------------- Marked as reviewed by bchristi (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/13564#pullrequestreview-1394947363
