Looks OK. Seems we were previously leaking (or at least not freeing) the
XineramaScreenInfo struct after we were done with it. -phil.
On 5/1/19 5:09 PM, Sergey Bylokhov wrote:
Hello.
Please review the fix for JDK 13.
Bug: https://bugs.openjdk.java.net/browse/JDK-8223045
Fix: http://cr.openjdk.java.net/~serb/8223045/webrev.01
This fix deletes the array which initialized on startup and contained
the screen resolution data.
After the fix these data calculated on-demand.
Note: It is possible that the list of screens(which we still cache,
see JDK-8076313[1]) can be out of sync to the real number of screens
on the system, for removed screens we will request the bounds of the
main screen if it fails we will request the screen bounds using X11.
[1] https://bugs.openjdk.java.net/browse/JDK-8076313
