RFR: 8280468: Crashes in getConfigColormap, getConfigVisualId, XVisualIDFromVisual on Linux [v5]

Maxim Kartashev duke at openjdk.java.net
Thu Apr 21 07:00:11 UTC 2022


On Fri, 8 Apr 2022 16:20:10 GMT, Phil Race <prr at openjdk.org> wrote:

>> Maxim Kartashev has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Restored the original code in X11GraphicsDevice.java that got auto-formatted
>
> src/java.desktop/unix/classes/sun/awt/X11/XCanvasPeer.java line 90:
> 
>> 88:                 getScreenDevices()[screenNum].
>> 89:                 getDefaultConfiguration();
>> 90:         }
> 
> why did you get rid of the just in case ?
> Actually why is this method being modified at all ? 
> I mean besides adding locking ..

@prrace polite ping

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

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



More information about the client-libs-dev mailing list