On Fri, 26 Sep 2025 10:17:59 GMT, Johan Sjölen <[email protected]> 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
