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 Sat, 10 Jan 2026 04:36:00 GMT, Kolby Moroz Liebl <duke at openjdk.org> wrote:
>> 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
Ok I applied the feedback
> I don't think this is needed once the isIOS() call is added? (which is added in this PR)
yeah it worked without it, updated the PR
I applied PlatformProps.TARGET_OS_IS_IOS to isIos
-------------
PR Review Comment: https://git.openjdk.org/mobile/pull/44#discussion_r2696591147
More information about the mobile-dev
mailing list