RFR: 8253702: BigSur java/lang/System/OsVersionTest.java: 10.16 != 11.0

Roger Riggs rriggs at openjdk.java.net
Thu Feb 11 17:48:41 UTC 2021


On Thu, 11 Feb 2021 17:38:42 GMT, Brian Burkhalter <bpb at openjdk.org> wrote:

>> src/java.base/macosx/native/libjava/java_props_macosx.c line 250:
>> 
>>> 248: 
>>> 249:         NSString *nsVerStr;
>>> 250:         // Copy out the char* if running on version other than pre-10.16 Mac OS (10.16 == 11.x)
>> 
>> Isn't the comment backwards from what the test does? I would think it should be "if running on version other than 10.16 Mac OS ...". Or am I missing something?
>
> @kevinrushforth I think you are correct. This is actually mea culpa I think as I had provided in a Slack thread a failed attempt at a patch for this which contained this comment.

So the words "other than" are too subtle?

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

PR: https://git.openjdk.java.net/jdk/pull/2530


More information about the core-libs-dev mailing list