On Thu, 16 Jul 2026 06:56:33 GMT, David Holmes <[email protected]> wrote:
>> src/hotspot/share/prims/jvmtiEnvBase.hpp line 35: >> >>> 33: #include "runtime/fieldDescriptor.hpp" >>> 34: #include "runtime/frame.hpp" >>> 35: #include "runtime/handles.inline.hpp" >> >> Do we need this include for the new Handle field on L539 below >> or can we get away with a forward decl on L46 below? > > This violates the rules - you don't include an inline.hpp into a plain .hpp Right, I moved the constructor definition to the cpp file and deleted this include. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/31759#discussion_r3598997898
