On Thu, 4 Sep 2025 14:12:50 GMT, Alexey Ivanov <[email protected]> wrote:
>> Khalid Boulanouare has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Organizes imports
>
> test/jdk/java/awt/Mixing/AWT_Mixing/GlassPaneOverlappingTestBase.java line 28:
>
>> 26: import java.awt.event.MouseAdapter;
>> 27: import java.awt.event.MouseEvent;
>> 28: import java.lang.Override;
>
> All the classes from `java.lang.*` are imported implicitly.
>
> That is, remove `import java.lang.Override`.
lang package imports removed.
> test/jdk/java/awt/Mixing/AWT_Mixing/GlassPaneOverlappingTestBase.java line
> 100:
>
>> 98: protected boolean isMultiFramesTest(){
>> 99: return false;
>> 100: }
>
> There should be no blank line before the closing brace of a method or
> constructor. That is the snippet should like this:
>
>
> public GlassPaneOverlappingTestBase(boolean defaultClickValidation) {
> super(defaultClickValidation);
> }
>
> @Override
> protected boolean isMultiFramesTest() {
> return false;
> }
Done.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25971#discussion_r2322618892
PR Review Comment: https://git.openjdk.org/jdk/pull/25971#discussion_r2322627072
