On Thu, 16 Mar 2023 06:32:30 GMT, Karthik P K <k...@openjdk.org> wrote:

>> modules/javafx.controls/src/test/java/test/javafx/scene/control/skin/LabelSkinTest.java
>>  line 2136:
>> 
>>> 2134:      *                                                                
>>>               *
>>> 2135:      
>>> *******************************************************************************/
>>> 2136: 
>> 
>> Minor: I would have used a javadoc explaining the purpose of this class 
>> instead of generic "Helper classes".
>
> Same comment as above. In addition to that, yes, updating the comment with 
> actual purpose of the class would be better. I will update that once we 
> decide if all the other comment blocks also should be changed or not.
> Since there was no comment for this class, I added the comment so that it 
> gets separated from "Tests for bug reports" tests.

Updated the comment with purpose of the class instead of generic comment.

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

PR: https://git.openjdk.org/jfx/pull/1059

Reply via email to