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/ Testing: jdk/submit + test/jdk/tools/jlink and test/jdk/jdk/modules tests on Linux x86_64 Thoughts? Thanks, Severin