Re: Removing unneeded imports from testcode

2020-01-14 Thread Kevin Rushforth
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

Removing unneeded imports from testcode

2020-01-14 Thread Robert Lichtenberger
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