<AWT Dev> Request for review: 7027045

Sergey Bylokhov Sergey.Bylokhov at oracle.com
Thu Aug 8 06:08:26 PDT 2013


On 08.08.2013 16:54, Konstantin Perikov wrote:
> Artem, could you answer on my last question?
>
>     P.S. Since, I'm newcomer, I don't have rights to push, so I need a
>     sponsor. Who could help me?
>
I will help to push these changes after review
>
> Thanks, Konstantin.
>
>
> 2013/8/8 Artem Ananiev <artem.ananiev at oracle.com 
> <mailto:artem.ananiev at oracle.com>>
>
>     Hi, Konstantin,
>
>     I looked through the changes, which mostly replacements
>     <code></code> with {@code}. This part of the fix looks fine.
>
>     Changing /* to /** for the "isInShow" field doesn't make sense, as
>     this field is transient and is not serialized anyway. However, it
>     doesn't hurt as well.
>
>     Please, keep comment about synchronization for the "type" field.
>     It's a hint for developers, that this field should only be
>     accessed or modified under the object lock. I agree, JavaDoc is
>     not the best place for such hints, but I don't see any better
>     solutions.
>
>     Non-technical comments:
>
>     1. Please, provide a direct link to webrev, so people can click
>     and see the changes. Downloading archives is not as convenient
>     (and in this particular case just impossible, as Yandex.Disk is in
>     Russian, people just won't be able to read the "Download" button).
>
>     2. Please, wait for at least one more person to have reviewed this
>     fix.
>
>     Thanks,
>
>     Artem
>
>     On 8/8/2013 2:20 PM, Konstantin Perikov wrote:
>
>         Hi, AWT team,
>
>         Could you please review the fix for the following bug:
>
>         *7027045:  : (doc) java/awt/Window.java has several typos in
>         javadoc*
>
>         Fix for version OpenJDK8. Also, I fix some <code></code> stuff
>         and change it for {@code }
>
>         The webrev is available here:
>
>         http://yadi.sk/d/KHB0hBlA7gkke
>
>
>         P.S. Since, I'm newcomer, I don't have rights to push, so I
>         need a sponsor. Who could help me?
>
>         Thanks,
>
>         Konstantin
>
>


-- 
Best regards, Sergey.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/awt-dev/attachments/20130808/461ef0a0/attachment.html 


More information about the awt-dev mailing list