RFR: 8364519: Sort share/classfile includes [v2]
Francesco Andreuzzi
duke at openjdk.org
Fri Aug 1 13:03:12 UTC 2025
On Fri, 1 Aug 2025 11:05:35 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
>> Francesco Andreuzzi has updated the pull request incrementally with one additional commit since the last revision:
>>
>> remove redundant includes
>
> src/hotspot/share/classfile/systemDictionary.cpp line 63:
>
>> 61: #include "oops/objArrayOop.inline.hpp"
>> 62: #include "oops/oop.hpp"
>> 63: #include "oops/oop.inline.hpp"
>
> Take this opportunity to remove the include of `oop.hpp`, since we are including `oop.inline.hpp` anyway?
Sure, I got a few here: 3477cbce67776546bbb0cbc0cf18302a6dab2c97
Could this be a possible improvement of `SortIncludes.java`?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26590#discussion_r2247927317
More information about the hotspot-dev
mailing list