RFR: 8339783: Implement JEP 479: Remove the Windows 32-bit x86 Port [v17]
Vladimir Kozlov
kvn at openjdk.org
Mon Nov 4 17:20:04 UTC 2024
On Mon, 4 Nov 2024 17:08:40 GMT, Magnus Ihse Bursie <ihse at openjdk.org> wrote:
>>> With that said, it is sure as heck confusing! Which also apparently Microsoft acknowledges by phasing in the term "Windows API". So I agree that we should try to rename everything currently called "win32" into "windows". But I'd rather see such a global rename refactoring, that will also affect the 64-bit Windows platforms, to be done as a separate, follow-up PR. Are you okay with that?
>>
>> Yes, I completely agree to do such clean up in separate RFE. Please, file one.
>
>> > With that said, it is sure as heck confusing! Which also apparently Microsoft acknowledges by phasing in the term "Windows API". So I agree that we should try to rename everything currently called "win32" into "windows". But I'd rather see such a global rename refactoring, that will also affect the 64-bit Windows platforms, to be done as a separate, follow-up PR. Are you okay with that?
>>
>> Yes, I completely agree to do such clean up in separate RFE. Please, file one.
>
> Good. I filed https://bugs.openjdk.org/browse/JDK-8343553.
@magicus
Back to my question about https://github.com/openjdk/jdk/blob/master/make/autoconf/flags-cflags.m4#L470 ?
I see only few uses of `#ifdef WIN32` in HotSpot which can be replaced with `#ifdef _WINDOWS`:
src/hotspot/share/runtime/sharedRuntimeTrans.cpp:#ifdef WIN32
src/hotspot/share/runtime/sharedRuntimeTrans.cpp:#ifdef WIN32
Note, there are a lot more usages of `WIN32` in JDK libraries native code which we may consider renaming later.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/21744#issuecomment-2455274589
More information about the core-libs-dev
mailing list