RFR: 8364087: Amend comment in globalDefinitions.hpp on "classfile_constants.h" include [v2]

duke duke at openjdk.org
Thu Jul 31 12:55:58 UTC 2025


On Thu, 31 Jul 2025 12:30:46 GMT, Francesco Andreuzzi <duke at openjdk.org> wrote:

>> The comment was introduced in [8231844](https://github.com/fandreuz/jdk/commit/fce43203699336e9513d06cd54b66735e93234a4). I looked around, and `<jvm.h>` is not included in any of the headers included by `globalDefinitions.hpp`, neither now nor in the original commit. Thus, I propose to amend the comment, as it [turned out to be confusing](https://github.com/openjdk/jdk/pull/26428/files#r2229575525) during an header clean up. 
>> 
>> Including `classfile_constants.h` is needed for e.g. `JVM_T_CHAR`, as stated by the comment, thus I'm going to leave that part intact.
>
> Francesco Andreuzzi has updated the pull request incrementally with one additional commit since the last revision:
> 
>   remove comment

@fandreuz 
Your change (at version 29231a33f7eda128088c2d0e8235868fc3c3b226) is now ready to be sponsored by a Committer.

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

PR Comment: https://git.openjdk.org/jdk/pull/26573#issuecomment-3139861080


More information about the hotspot-dev mailing list