<Swing Dev> [9] Review request for 8057184 JCK8's api/javax_swing/JDesktopPane/descriptions.html#getset failed with GTKLookAndFeel on Linux and Solaris run v.s. JDK8+
Alexander Scherbatiy
alexandr.scherbatiy at oracle.com
Thu Sep 4 13:32:00 UTC 2014
On 9/4/2014 4:24 PM, Anton V. Tarasov wrote:
> Hi Alexander,
>
> Looks fine for me, however you could use HashSet instead of ArrayList
> in order to avoid the check for "contains".
>
Could you review the updated fix:
http://cr.openjdk.java.net/~alexsch/8057184/webrev.01
I changed the ArrayList to LinkedHashSet just to preserve the order
of added internal frames.
The javadoc does not require it but it preserves the previous behavior.
Thanks,
Alexandr.
> 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.
>>
>
More information about the swing-dev
mailing list