<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">Hi Sergey,<div class=""><br class=""></div><div class="">Thanks for the comments. Here is updated version of the fix:</div><div class=""><br class=""></div><div class="">Webrev: <a href="http://cr.openjdk.java.net/~avu/JDK-8197499/webrev.02/" class="">http://cr.openjdk.java.net/~avu/JDK-8197499/webrev.02/</a></div><div class=""><br class=""></div><div class="">Best Regards,</div><div class="">Alexey<br class=""><div><br class=""><blockquote type="cite" class=""><div class="">On 12 Feb 2018, at 21:29, Sergey Bylokhov <<a href="mailto:Sergey.Bylokhov@oracle.com" class="">Sergey.Bylokhov@oracle.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div class="">Hi, Alexey.<br class="">Two small comments:<br class=""> - It looks like that after the fix we will reset the value to "null" even if it was set by the user, for example the user may want to minimize the memory footprint if he knows that application limite the size of the windows.<br class=""> - setDoubleBufferMaximumSize includes clearImages() so it can be removed from displayChanged();<br class=""><br class="">On 12/02/2018 02:12, Alexey Ushakov wrote:<br class=""><blockquote type="cite" class="">Thanks, for the catch, Laurent!<br class="">I forgot to remove my debug output that I used to detect the problem and verify the fix. It might be useful for reviewers though, so I left the original webrev as is and added the new one.<br class="">Webrev: <a href="http://cr.openjdk.java.net/~avu/JDK-8197499/webrev.01/" class="">http://cr.openjdk.java.net/~avu/JDK-8197499/webrev.01/</a><br class="">Best Regards,<br class="">Alexey<br class=""><blockquote type="cite" class="">On 12 Feb 2018, at 12:14, Laurent Bourgès <<a href="mailto:bourges.laurent@gmail.com" class="">bourges.laurent@gmail.com</a> <<a href="mailto:bourges.laurent@gmail.com" class="">mailto:bourges.laurent@gmail.com</a>>> wrote:<br class=""><br class="">Hi,<br class=""><br class="">I am not an official reviewer.<br class=""><br class="">I just noted you left a stdout statement that should be removed or commented out (trace).<br class=""><br class="">You fix seems trivial<br class=""><br class="">Laurent<br class=""><br class="">Le 12 févr. 2018 09:14, "Alexey Ushakov" <<a href="mailto:alexey.ushakov@jetbrains.com" class="">alexey.ushakov@jetbrains.com</a> <<a href="mailto:alexey.ushakov@jetbrains.com" class="">mailto:alexey.ushakov@jetbrains.com</a>>> a écrit :<br class=""><br class=""> Hello,<br class=""><br class=""> Here is the fix of the RepaintManager that adjust maximum double<br class=""> buffer size after changes in the display environment. The fix is<br class=""> targeted for openjdk10 but the problem exists in previous<br class=""> releases. Please, have a look.<br class=""><br class=""> Bug: <a href="https://bugs.openjdk.java.net/browse/JDK-8197499" class="">https://bugs.openjdk.java.net/browse/JDK-8197499</a><br class=""> <<a href="https://bugs.openjdk.java.net/browse/JDK-8197499" class="">https://bugs.openjdk.java.net/browse/JDK-8197499</a>><br class=""> Webrev: <a href="http://cr.openjdk.java.net/~avu/JDK-8197499/webrev.00/" class="">http://cr.openjdk.java.net/~avu/JDK-8197499/webrev.00/</a><br class=""> <<a href="http://cr.openjdk.java.net/~avu/JDK-8197499/webrev.00/" class="">http://cr.openjdk.java.net/~avu/JDK-8197499/webrev.00/</a>><br class=""><br class=""> Best Regards,<br class=""> Alexey <br class=""></blockquote></blockquote><br class=""><br class="">-- <br class="">Best regards, Sergey.<br class=""></div></div></blockquote></div><br class=""></div></body></html>