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
