On 20/05/2019 10:05, Phil Race wrote:
Looks OK. Seems we were previously leaking (or at least not freeing) the
XineramaScreenInfo struct after we were done with it. -phil.
Correct.
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
--
Best regards, Sergey.
