Hi Alexander,Looks fine for me, however you could use HashSet instead of ArrayList in order to avoid the check for "contains".
Thanks, Anton. On 04.09.2014 15:29, Alexander Scherbatiy wrote:
Hello, Could you review the fix: bug: https://bugs.openjdk.java.net/browse/JDK-8057184 webrev: http://cr.openjdk.java.net/~alexsch/8057184/webrev.00 The fix skips the same internal frames from the JDesktopPane.getAllFrames() results. Thanks, Alexandr.