[lworld] RFR: 8378186: [lworld] Assign default values and minor cleanup [v2]

Coleen Phillimore coleenp at openjdk.org
Thu Feb 19 18:11:51 UTC 2026


On Thu, 19 Feb 2026 14:35:25 GMT, Dan Heidinga <heidinga at openjdk.org> wrote:

>> This provides a default value for some locals so it's obvious they are always set and does some minor cleanup.
>
> Dan Heidinga has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Review comments

src/hotspot/share/classfile/stackMapFrame.cpp line 55:

> 53:    * an error has already been determined.  This just provides a safe
> 54:    * obvious fallback value.
> 55:    */

I'm sorry I have an additional comment.  This should be // comments not /* per hotspot coding convention.

-------------

PR Review Comment: https://git.openjdk.org/valhalla/pull/2130#discussion_r2829380556


More information about the valhalla-dev mailing list