Review Request: JDK-8164689: Retrofit jar, jlink, jmod as a ToolProvider
Alan.Bateman at oracle.com
Wed Oct 12 06:07:59 UTC 2016
On 11/10/2016 19:03, Mandy Chung wrote:
> This patch updates jar, jlink, jmod tool to be a provider of the new tool SPI. Some tests are also updated to replace the use of internal APIs with ToolProvider::findFirst to look up a tool provider. There are more tests that can be updated and something to be cleaned up in the future.
This update to the tools looks good. For the tests then
ToolProvider.findFirst("jar").get() will draw the attention of the
optional police and might be better to use orElseThrow to supply an
exception that clearly indicates that the tool cannot be found.
More information about the core-libs-dev