On Tue, 18 Aug 2026 14:45:15 GMT, Matthias Baesken <[email protected]> wrote:

> sun/awt/OSInfo.java contains very old Windows versions like Windows 95,98, 
> ME. Those can be removed.
> 
> ---------
> - [x] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

Btw. the whole part

if (!isOnWindows7()) {
 ...
}

https://github.com/openjdk/jdk/blob/45f3ef3ce2f9aa1bc54359c92bfd267a4f5b26a8/src/java.desktop/windows/classes/com/sun/java/swing/plaf/windows/WindowsLookAndFeel.java#L1677

is most likely obsolete these days.

src/java.desktop/windows/classes/com/sun/java/swing/plaf/windows/WindowsLookAndFeel.java
 line 1587:

> 1585: 
> 1586:     static boolean isOnWindows7() {
> 1587:         return OSInfo.getOSType() == OSInfo.OSType.WINDOWS

btw, where are these days always at least on Windows 7, so why not just return 
true in this method ?

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

PR Comment: https://git.openjdk.org/jdk/pull/32419#issuecomment-5341696559
PR Review Comment: https://git.openjdk.org/jdk/pull/32419#discussion_r3812652555

Reply via email to