Can I please get a review of this change which does a tiny cleanup in the 
macosx specific code in `java_props_macosx.c`?

As noted in https://bugs.openjdk.org/browse/JDK-8342647 the 
`setOSNameAndVersion` function used to dynamically call 
`NSProcessInfo.operatingSystemVersion` because that property wasn't 
(statically) available until macosx 10.10 version. Since that version this 
property has been available even in the latest macosx versions 
https://developer.apple.com/documentation/foundation/nsprocessinfo/1410906-operatingsystemversion,
 so it's now possible to statically reference it in the code.

The change in this PR replaces the use of `NSInvocation` with the static 
reference to this property. No new tests have been added. Existing tier1, tier2 
and tier3 tests with this change have all passed.

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

Commit messages:
 - 8342647: [macosx] Clean up the NSInvocation based call to 
NSProcessInfo.operatingSystemVersion

Changes: https://git.openjdk.org/jdk/pull/21595/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=21595&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8342647
  Stats: 42 lines in 1 file changed: 3 ins; 22 del; 17 mod
  Patch: https://git.openjdk.org/jdk/pull/21595.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/21595/head:pull/21595

PR: https://git.openjdk.org/jdk/pull/21595

Reply via email to