On Thu, 24 Nov 2022 14:05:41 GMT, Andrew Haley <a...@openjdk.org> wrote:
>> JEP 429 implementation.
>
> Andrew Haley has updated the pull request incrementally with one additional
> commit since the last revision:
>
> Unused variable
src/hotspot/share/prims/jvm.cpp line 1385:
> 1383: vframeStream vfst(thread);
> 1384: for(; !vfst.at_end(); vfst.next()) {
> 1385: int loc = 0;
Use -1 instead (see below)?
src/hotspot/share/prims/jvm.cpp line 1400:
> 1398: }
> 1399:
> 1400: if (loc != 0) {
As 0 is normally a valid local number, how about using -1 to mean "not found"?
-------------
PR: https://git.openjdk.org/jdk/pull/10952
