On Mon, 23 Aug 2021 17:01:46 GMT, Phil Race <p...@openjdk.org> wrote:

>> Pankaj Bansal has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Review Comments
>
> src/java.desktop/share/classes/javax/swing/JComponent.java line 188:
> 
>> 186:  * change according to look and feel, as some look and feels may add a
>> 187:  * default component to a container, where as other look and feels may
>> 188:  * choose not to do so.
> 
> "a" is not "any number" so you need to make your wording consistent.
> Also I think you need to say that even for a specific L&F it may vary 
> depending on configuration properties of the 
> JComponent.
> 
> And perhaps finish with
> "In summary, it is not valid to assume a JComponent has no children just 
> because the application did not directly add them"  ?

Done

-------------

PR: https://git.openjdk.java.net/jdk/pull/5121

Reply via email to