RFR: 8313396: Portable implementation of FORBID_C_FUNCTION and ALLOW_C_FUNCTION [v2]

David Holmes dholmes at openjdk.org
Fri Jan 10 07:18:37 UTC 2025


On Thu, 9 Jan 2025 06:31:55 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:

>> Yes, I think it's fine to say !WINDOWS instead of listing all the posix ports.  We've been avoiding dispatch files and once it reaches a threshold of too many #ifdef !WINDOWS #include posix.hpp one, then we could add another macro like the OS_CPU one.
>> 
>> Also the copyright script added 2025 for these because they started with 2024 so it's sort of a bug in the script but not really solvable because it doesn't know that you didn't check this in in 2024.
>
> @coleenp The copyrights on the new files being 2024-2025 is intentional. They were first published (in this PR)
> in 2024, so I think are supposed to have that starting year.

@kimbarrett I understand. I also don't like the includer needing to know about this, and maybe it is time to add OS_FAMILY_HEADER to deal with it. But in the absence of the new macro I prefer this break of abstraction to the creation of a bunch of tiny forwarding files.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22890#discussion_r1909925627


More information about the hotspot-dev mailing list