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.


Reply via email to