Re: [9] Review Request: 8151857 [TEST_BUG] bug6544309.java fails intermittently

+1 (not a reviewer) Thanks, Alexander On 3/15/2016 10:53 AM, Yuri Nesterenko wrote: Looks fine. -yan On 03/15/2016 03:28 AM, Sergey Bylokhov wrote: Hello, Please review the small fix for jdk9. The test javax/swing/JPopupMenu/6544309/bug6544309.java fails if executed in a batch via jtreg. Th

Re: RFR JDK-8133134: docs: replace tags (obsolete in html5) for java.desktop

Thanks! On 8/12/2015 11:00 AM, Alexander Scherbatiy wrote: The fix looks good to me. Thanks, Alexandr. On 8/11/2015 6:45 PM, Alexander Stepanov wrote: Hello, Could you please review the following fix: http://cr.openjdk.java.net/~avstepan/8133134/webrev.00 for https

RFR JDK-8133134: docs: replace tags (obsolete in html5) for java.desktop

Hello, Could you please review the following fix: http://cr.openjdk.java.net/~avstepan/8133134/webrev.00 for https://bugs.openjdk.java.net/browse/JDK-8133134 Deprecated tags replaced with {@code}. Thanks, Alexander

Re: RFR [9] 8081313: MultipleDocumentHandling.java: tidy warnings

Thanks! On 03.06.2015 15:04, Alexander Scherbatiy wrote: The fix looks good to me. Thanks, Alexandr. On 5/29/2015 2:58 PM, alexander stepanov wrote: Hello, Could you please review fix http://cr.openjdk.java.net/~avstepan/8081313/webrev.00/ for https://bugs.openjdk.java.net/browse/JDK

Re: RFR 8077036: swing docs: fix some tidy warnings

Thanks! On 03.06.2015 15:16, Alexander Scherbatiy wrote: The fix looks good to me. Thanks, Alexandr. On 6/3/2015 2:29 PM, alexander stepanov wrote: Hello, Could you please review the following fix http://cr.openjdk.java.net/~avstepan/8077036/webrev.00/ for https

RFR 8077036: swing docs: fix some tidy warnings

Hello, Could you please review the following fix http://cr.openjdk.java.net/~avstepan/8077036/webrev.00/ for https://bugs.openjdk.java.net/browse/JDK-8077036 Just some minor docs cleanup; no other code affected. Thanks, Alexander

RFR [9] 8081313: MultipleDocumentHandling.java: tidy warnings

Sorry; was sent to wrong mailing list initially. On 29.05.2015 14:58, alexander stepanov wrote: Hello, Could you please review fix http://cr.openjdk.java.net/~avstepan/8081313/webrev.00/ for https://bugs.openjdk.java.net/browse/JDK-8081313 Just a minor fix of docs; other code is not affected

RFR [9] 8081313: MultipleDocumentHandling.java: tidy warnings

Hello, Could you please review fix http://cr.openjdk.java.net/~avstepan/8081313/webrev.00/ for https://bugs.openjdk.java.net/browse/JDK-8081313 Just a minor fix of docs; other code is not affected. Thanks, Alexander

Re: [9] Review Request for 8046445: Fix doclint warnings for javax.swing.plaf.multi

Thanks! On 19.06.2014 18:32, Alexander Scherbatiy wrote: The fix looks good for me. Thanks, Alexandr. On 6/16/2014 3:45 PM, alexander stepanov wrote: Hello, Could you please review the fix for the following bug: https://bugs.openjdk.java.net/browse/JDK-8046445 Webrev corresponding

[9] Review Request for 8046445: Fix doclint warnings for javax.swing.plaf.multi

Hello, Could you please review the fix for the following bug: https://bugs.openjdk.java.net/browse/JDK-8046445 Webrev corresponding: http://cr.openjdk.java.net/~avstepan/8046445/webrev.00/ Just a minor cleanup of javadoc to avoid doclint warnings. Thanks, Alexander

Re: [9] Review Request for 8042120: Fix doclint warnings from javax.swing.text.html.parser

thanks! On 16.05.2014 17:10, Alexander Scherbatiy wrote: The fix looks good for me. Thanks, Alexandr. On 5/16/2014 1:21 PM, alexander stepanov wrote: Hello Alexander, Removed; please find the the updated webrev here: http://cr.openjdk.java.net/~yan/8042120/webrev.01/ Thanks

Re: [9] Review Request for 8042120: Fix doclint warnings from javax.swing.text.html.parser

with the message + * (err + " " + arg1 + " " + arg2 + " " + arg3). Could you remove this Comment? The method implementation can be changed and it will require additional CCC request for the javadoc change. Thanks, Alexandr. On 4/29/2014 6:32 PM, alexander step

[9] Review Request for 8042120: Fix doclint warnings from javax.swing.text.html.parser

Hello, Could you please review the fix for the following bug: https://bugs.openjdk.java.net/browse/JDK-8042120 Webrev corresponding: http://cr.openjdk.java.net/~yan/8042120/webrev.00/ Just a minor cleanup of javadoc to avoid doclint warnings. Thanks, Alexander

Re: [9] Review Request for 8039074: tidy warnings cleanup for javax.swing

27;t you add a real summary for the table? src/share/classes/javax/swing/plaf/synth/doc-files/synthFileFormat.html - again, why don't you add a real summary? With best regards. Petr. On 02.04.2014, at 16:35, alexander stepanov wrote: Hello, Could you please review the fix for the followi

Re: [9] Review Request for 8039074: tidy warnings cleanup for javax.swing

vax/swing/plaf/synth/doc-files/synthFileFormat.html - again, why don't you add a real summary? With best regards. Petr. On 02.04.2014, at 16:35, alexander stepanov wrote: Hello, Could you please review the fix for the following bug: https://bugs.openjdk.java.net/browse/JDK-8039074 Webrev c

[9] Review Request for 8039074: tidy warnings cleanup for javax.swing

Hello, Could you please review the fix for the following bug: https://bugs.openjdk.java.net/browse/JDK-8039074 Webrev corresponding: http://cr.openjdk.java.net/~yan/8039074/webrev.00/ Just a minor cleanup of javadoc to avoid tidy warnings; no other code affected. Thanks.

[8] Review request for 8025070: [javadoc] fix some javadoc errors in javax/swing/plaf/synth

Hello, Could you please review the fix for the following bug: https://bugs.openjdk.java.net/browse/JDK-8025070 webrev corresponding: http://cr.openjdk.java.net/~alexsch/alexander-stepanov/webrev.00/ The patch contains only simple markup fixes; no changes/fixes in documentation text; the