On 7/16/19 6:48 AM, Claes Redestad wrote:
Hi,

refactored to use a BootLoader::loadLibrary API that is only visible
within the java.base module:

http://cr.openjdk.java.net/~redestad/8227587/open.03/


Looks good.

Nit:  in JavaLangAccess
   321     void loadLibrary(Class<?> klass, String library);

s/klass/caller/

I've retained the bridge to ClassLoader::loadLibrary for performance,
but without any magic or privilege escalation involved. Moving sys_path
/ systemNativeLibraries out of ClassLoader seems quite complicated, so
I've not attempted that refactoring for this RFE.

Yes, it is non-trivial.  I will file a RFE for this possible clean up.

Mandy

Reply via email to