<AWT Dev> [9] Review request for 8044157: improve recently submitted AWT_Mixing tests
Petr Pchelko
petr.pchelko at oracle.com
Wed Jun 4 08:44:31 UTC 2014
Hello, Yuri.
A couple of comments:
1. In HierarchyBoundsListenerMixingTest.java a couple of lines is left commented out. What's going on there? Same with MixingPanelsResizing.java
2. I see that tests open a TestDialog with some useless instructions and obvious info. What do you think about removing it completely together with the dialog code?
With best regards. Petr.
On 04 июня 2014 г., at 12:23, Yuri Nesterenko <yuri.nesterenko at oracle.com> wrote:
> Weekly friendly reminder!
>
> Best regards,
> -yan
>
> On 05/28/2014 07:06 PM, Yuri Nesterenko wrote:
>> Hi,
>>
>> please review this change for RFE
>> https://bugs.openjdk.java.net/browse/JDK-8044157
>>
>> Webrev is in
>> http://cr.openjdk.java.net/~yan/8044157/webrev.00
>>
>> Thanks,
>> -yan
>> _______________
>> In this change, we remove an exact copy of one helper class
>> and change some stray System.exits to RuntimeExceptions.
>>
>> Tested on 2 different Linux systems, on OS X and
>> 2 Windows (7, 8.1).
>
More information about the awt-dev
mailing list