On Fri, 25 Sep 2020 06:06:31 GMT, Kim Barrett <kbarr...@openjdk.org> wrote:

>> Please review this small patch to enable the OSX build using Xcode 12.0.
>> 
>> Thanks,
>> Paul
>
> src/java.desktop/macosx/native/libawt_lwawt/awt/CSystemColors.m line 129:
> 
>> 127:     NSColor* result = nil;
>> 128:
>> 129:     if (colorIndex < ((useAppleColor) ? 
>> sun_lwawt_macosx_LWCToolkit_NUM_APPLE_COLORS :
>> java_awt_SystemColor_NUM_COLORS)) {
> 
> This looks like a plain old bug fix, nothing really to do with the compiler 
> upgrade.  The new compiler presumably just
> has a new warning that brought attention to the problem.  As such, I don't 
> think it belongs in a PR that's about issues
> related to the compiler upgrade.  Someone might want to backport just this 
> fix, for example.

Hello  Kim, Paul -  so should we open a separate bug for the
src/java.desktop/macosx/native/libawt_lwawt/awt/CSystemColors.m issue because 
it might be a general  problem just
uncovered by the new compiler ? Paul , do you want to do this ? Or should I 
open a bug and post a separate change for
the useAppleColor check in CSystemColors.m ?

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

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

Reply via email to