[rfc][icedtea-web] Extract JNLPClassLoader#CodeBaseClassLoader as non-inner class

Jiri Vanek jvanek at redhat.com
Tue Oct 23 02:08:11 PDT 2012


On 10/02/2012 09:40 PM, Adam Domurad wrote:
> On 10/02/2012 03:38 PM, Adam Domurad wrote:
>> JNLPClassLoader.java needs some components broken off, and this is an easy start.
>>
>> This changes no functionality, the class has simply been moved and references updated.
>> Note that the class can not be made package-private as would be ideal, because of a reference in PluginAppletViewer.java.
>>
>> ChangeLog:
>> 2012-10-02 Adam Domurad <adomurad at redhat.com>
>>
>> Extract out JNLPClassLoader#CodeBaseClassLoader as a non-inner class.
>> * netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.java: Extracted CodeBaseClassLoader out.
>> * netx/net/sourceforge/jnlp/runtime/CodeBaseClassLoader.java: New, put
>> CodeBaseClassLoader here.
>> * netx/net/sourceforge/jnlp/runtime/JNLPSecurityManager.java: Updated
>> CodeBaseClassLoader reference to correct location.
>> * plugin/icedteanp/java/sun/applet/PluginAppletViewer.java: Updated
>> CodeBaseClassLoader reference to correct location.
>> * tests/netx/unit/net/sourceforge/jnlp/runtime/CodeBaseClassLoaderTest.java:
>> Updated CodeBaseClassLoader reference to correct location.
>>
> Apologies hit send prematurely, patch attached now.

I agree with this refactoring.

J.



More information about the distro-pkg-dev mailing list