Removing unneeded imports from testcode

Kevin Rushforth kevin.rushforth at oracle.com
Tue Jan 14 11:51:38 UTC 2020


As a general rule, we avoid unrelated fixes. If your fix needed to add 
any new import, then I don't mind removing unused ones at the same time. 
If not, then you can either leave it as-is or file a cleanup issue.

-- Kevin


For test fixes, I don't mind

On 1/14/2020 3:40 AM, Robert Lichtenberger wrote:
> I'm in the process of preparing a fix for JDK-8236912 and I (or rather
> eclipse) have noticed that WebViewTest contains an import of
> java.util.concurrent.FutureTask that is not needed.
> What is the correct way of handling such things:
> * Including them into the Pull-Request for the fix of JDK-8236912
> * Leaving the unneeded import as-is.
> * Opening another issue
> Best regards,
> Robert



More information about the openjfx-dev mailing list