RFR: 8367656: Refactor Constantpool's operand array into two [v7]

Serguei Spitsyn sspitsyn at openjdk.org
Wed Oct 8 20:36:25 UTC 2025


On Fri, 26 Sep 2025 10:17:59 GMT, Johan Sjölen <jsjolen at openjdk.org> wrote:

>> Why not? This PR is introducing these two classes. It is better to do in a simplified form. :)
>
> It's only introducing the BSMAttributeEntries class, the other one is pre-existing. I don't want the PR to get stuck on whether or not to do a rename of something previously determined to be fine.

Okay then. I agree, it is not a good idea to rename pre-existing identifiers in this PR. But naming consistency is still important.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/27198#discussion_r2414971254


More information about the hotspot-dev mailing list