Re: Review request for 8042835: unexpected mnemonic in JFileChooser

2014-08-12 Thread artem malinko
Thank you, Alex. On 8/12/2014 4:59 PM, Alexander Potochkin wrote: Looks good! Thanks alexp On 8/11/2014 11:39 AM, artem malinko wrote: Hello again. Could somebody else join to the review please? On 8/7/2014 6:21 PM, artem malinko wrote: Thank you, Alexandr. > The link to the webrev points

Re: Review request for 8042835: unexpected mnemonic in JFileChooser

2014-08-12 Thread Alexander Potochkin
Looks good! Thanks alexp On 8/11/2014 11:39 AM, artem malinko wrote: Hello again. Could somebody else join to the review please? On 8/7/2014 6:21 PM, artem malinko wrote: Thank you, Alexandr. > The link to the webrev points to the 8047288 fix instead of 8042835 Sorry, some weird copy and pas

Re: Review request for 8042835: unexpected mnemonic in JFileChooser

2014-08-11 Thread artem malinko
Hello again. Could somebody else join to the review please? On 8/7/2014 6:21 PM, artem malinko wrote: Thank you, Alexandr. > The link to the webrev points to the 8047288 fix instead of 8042835 Sorry, some weird copy and paste. Here is the good link for the next reviewer: http://cr.openjdk.java

Re: Review request for 8042835: unexpected mnemonic in JFileChooser

2014-08-07 Thread artem malinko
Thank you, Alexandr. > The link to the webrev points to the 8047288 fix instead of 8042835 Sorry, some weird copy and paste. Here is the good link for the next reviewer: http://cr.openjdk.java.net/~dmarkov/8042835/webrev.00/ On 8/7/2014 6:04 PM, Alexander Scherbatiy wrote: The fix looks good

Re: Review request for 8042835: unexpected mnemonic in JFileChooser

2014-08-07 Thread Alexander Scherbatiy
The fix looks good to me. The link to the webrev points to the 8047288 fix instead of 8042835. Thanks, Alexandr. On 8/6/2014 4:04 PM, artem malinko wrote: Hello, Swing team. Please review a fix for the issue: https://bugs.openjdk.java.net/browse/JDK-8042835 The fix is available at: http:

Review request for 8042835: unexpected mnemonic in JFileChooser

2014-08-06 Thread artem malinko
Hello, Swing team. Please review a fix for the issue: https://bugs.openjdk.java.net/browse/JDK-8042835 The fix is available at: http://cr.openjdk.java.net/~dmarkov/8042835/webrev.00/ Bug relates to: https://bugs.openjdk.java.net