RFR: 8289770: Remove Windows version macro from ShellFolder2.cpp
Sergey Bylokhov
serb at openjdk.org
Tue Apr 16 07:00:42 UTC 2024
On Thu, 11 Apr 2024 09:33:09 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:
> This clean-up PR removes unused Windows version macro from `ShellFolder2.cpp`.
>
> `IS_WINVISTA` was not used at all.
>
> `IS_WINXP` guarded support for icons with alpha channel. It is now safe to assume Java runs on a Windows version later than Windows XP. Java launchers specify 6.0 as the minimum OS version which corresponds to Windows Vista.
src/java.desktop/windows/native/libawt/windows/ShellFolder2.cpp line 131:
> 129: #ifndef IS_WINVISTA
> 130: #define IS_WINVISTA (!(::GetVersion() & 0x80000000) && LOBYTE(LOWORD(::GetVersion())) >= 6)
> 131: #endif
there are exactly the same macro in awt.h which are mostly unused except "IS_WIN8"
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/18736#discussion_r1566819367
More information about the client-libs-dev
mailing list