On 28/01/2019 15:00, Severin Gehwolf wrote:
Hi,
There seems to be dead code in class TaskHelper. Plugins are loaded via
ServiceLoader from the module boot loader. I don't see how this code
could ever be reached. The proposal is to remove it for clarity.
Bug: https://bugs.openjdk.java.net/browse/JDK-8217877
webrev: http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8217877/webrev.01/
This code was used in JDK 9 for the experimental plugin interface, it
was disabled near the end of the release. So I think it's okay to remove
this code, meaning your patch looks okay, but it might of course come
back when that plugin interface is looked at again.
-Alan
