RFR: 8328749: Remove unused imports in javafx.web [v2]

Hima Bindu Meda hmeda at openjdk.org
Tue Mar 26 07:03:27 UTC 2024


On Fri, 22 Mar 2024 16:11:52 GMT, Andy Goryachev <angorya at openjdk.org> wrote:

>> Using Eclipse IDE to remove unused imports **javafx.web** module, and update the copyright year to 2024. Using wildcard for more than 10 static imports.
>> 
>> 
>> --
>> 
>> This is a trivial change, 1 reviewer is probably enough.
>
> Andy Goryachev has updated the pull request incrementally with one additional commit since the last revision:
> 
>   removed swing file

modules/javafx.web/src/main/native/Source/WebCore/bindings/java/dom3/java/com/sun/webkit/dom/DOMWindowImpl.java line 37:

> 35: import org.w3c.dom.views.AbstractView;
> 36: import org.w3c.dom.views.DocumentView;
> 37: import com.sun.webkit.Disposer;

Minor: Generally, imports are added in alphabetic order.

modules/javafx.web/src/main/native/Source/WebCore/bindings/java/dom3/java/com/sun/webkit/dom/NodeImpl.java line 38:

> 36: import org.w3c.dom.events.EventListener;
> 37: import org.w3c.dom.events.EventTarget;
> 38: import com.sun.webkit.Disposer;

Minor: Generally, imports are added in alphabetic order.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1421#discussion_r1538678437
PR Review Comment: https://git.openjdk.org/jfx/pull/1421#discussion_r1538678727


More information about the openjfx-dev mailing list