RFR: 8159855: Create an SPI for tools

Alan Bateman Alan.Bateman at oracle.com
Wed Oct 5 14:34:26 UTC 2016


On 05/10/2016 05:19, Mandy Chung wrote:

> :
>
> ToolProvider::findFirst(String name) can find tool providers on classpath.  I think it needs to wrap the for-loop (specifically iterating on providers) with doPrivileged due to the stack-based permission check.
>
Yes, that will be needed, otherwise the caller will need permission to 
get the system class loader and whatever other permissions are needed to 
locate and and load the tools.

The rest looks okay to me. Personally I would space out the javadoc tags 
more, and indent the second/third lines more than one space but that is 
just minor stuff.

-Alan


More information about the core-libs-dev mailing list