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