On Tue, 13 Oct 2026 06:53:54 GMT, Serguei Spitsyn <[email protected]> wrote:

>> Stefan Karlsson has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   Update cdsProtectionDomain.cpp
>
> src/hotspot/share/cds/cdsProtectionDomain.cpp line 39:
> 
>> 37: #include "memory/universe.hpp"
>> 38: #include "oops/instanceKlass.hpp"
>> 39: #include "oops/oopCast.inline.hpp"
> 
> Nit: It is better to add `#include "oops/oop.inline.hpp"` instead of the 
> removed `"oops/oopCast.inline.hpp"`.
> Other than this the fix looks good to me.

Good catch! I'll fix that.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/33287#discussion_r4256024620

Reply via email to