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