> Fixed. No you didn't :) > http://cr.openjdk.java.net/~yan/8049808/webrev.01/ Looks like you've uploaded the exact same webrev.
With best regards. Petr. On 10 июля 2014 г., at 16:58, andrei.eremeev <[email protected]> wrote: > Fixed. > http://cr.openjdk.java.net/~yan/8049808/webrev.01/ > > Andrei > > On 07/10/2014 04:03 PM, Petr Pchelko wrote: >> Hello, Andrei. >> >> BasicBorders:353 - typo, the constructor constructs and not draws. >> BasicBorders:442 - need an empty line >> BasicListUI - what the {JList} construct is intended to mean? You forgot >> @code? >> BasicListUI:247 - forgot @code >> BasicListUI:1621 - same here >> BasicMenuItemUI:481 @return what? >> >> With best regards. Petr. >> >> On 10 июля 2014 г., at 15:29, andrei.eremeev <[email protected]> >> wrote: >> >>> Hi Swing team, >>> >>> Please, review the fix for the issue: >>> https://bugs.openjdk.java.net/browse/JDK-8049808 >>> >>> The fix is available at: >>> http://cr.openjdk.java.net/~yan/8049808/ >>> >>> >>> Andrei >
