RFR: 8367656: Refactor Constantpool's operand array into two [v12]
Serguei Spitsyn
sspitsyn at openjdk.org
Wed Oct 15 18:01:45 UTC 2025
On Wed, 15 Oct 2025 17:28:02 GMT, Johan Sjölen <jsjolen at openjdk.org> wrote:
>> src/hotspot/share/oops/constantPool.inline.hpp line 90:
>>
>>> 88: return resolved_references()->obj_at(cache()->resolved_method_entry_at(index)->resolved_references_index());
>>> 89: }
>>> 90:
>>
>> Nit: Is this really needed?
>
> I believe that the style is to have an empty line between the code and the endif, so this is a style fix.
The issue is this is the only fix in this file. Should we go and fix all style issues around?
In fact, I'm okay with this tweak. :)
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/27198#discussion_r2433497456
More information about the serviceability-dev
mailing list