<AWT Dev> Request for review: 7027045

Artem Ananiev artem.ananiev at oracle.com
Thu Aug 8 05:33:47 PDT 2013


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
>


More information about the awt-dev mailing list