RFR: 8374522: Mobile: Add iOS as a platform in OperatingSystem.java
Kolby Moroz Liebl
duke at openjdk.org
Sun Feb 1 22:48:41 UTC 2026
On Wed, 7 Jan 2026 12:10:08 GMT, Magnus Ihse Bursie <ihse at openjdk.org> wrote:
>> src/java.base/share/classes/jdk/internal/util/OperatingSystem.java line 106:
>>
>>> 104: @ForceInline
>>> 105: public static boolean isMacOS() {
>>> 106: // Treat iOS as macOS for compatibility with existing libraries unless specific IOS checks exist
>>
>> I don't think this is needed once the isIOS() call is added? (which is added in this PR)
>
> Also, it looks really odd. Why isn't it something like:
>
> return current() == MACOS || current() == IOS;
>
> or
>
> return PlatformProps.TARGET_OS_IS_MACOS || PlatformProps.TARGET_OS_IS_IOS;
>
> ?
I will test this tomorrow and report back and go from there
-------------
PR Review Comment: https://git.openjdk.org/mobile/pull/44#discussion_r2678314189
More information about the mobile-dev
mailing list