RFR: 8366331: Sort share/prims includes [v3]
Francesco Andreuzzi
duke at openjdk.org
Thu Aug 28 13:01:39 UTC 2025
On Thu, 28 Aug 2025 09:31:42 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
>> Francesco Andreuzzi has updated the pull request incrementally with one additional commit since the last revision:
>>
>> nn
>
> src/hotspot/share/prims/foreignGlobals.cpp line 25:
>
>> 23:
>> 24: #include "classfile/javaClasses.hpp"
>> 25: #include "foreignGlobals.hpp"
>
> This does not look right, even in the original code. Isn't it the same as `prims/foreignGlobals.hpp`, which is subsumed by the existing `prims/foreignGlobals.inline.hpp`?
Right, removed in b0d163391270189772939ef2849e45dbf5963063
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26980#discussion_r2307338536
More information about the hotspot-dev
mailing list