RE: [cp-patches] FYI: Patch ClassLoader & VMClassLoader

[Jeroen Frijters]

Michael Koch wrote:
> Am Montag, 18. Oktober 2004 17:24 schrieb Jeroen Frijters:
> > Michael Koch wrote:
> > > Can you please tell us why this patch is needed ?
> >
> > The original implementation of VMClassLoader.getSystemClassLoader
> > was completely broken. I posted an RFC on this patch a week ago.
> 
> I read that RFC but you only said its broken, but not what was broken.
> I just ask because I look into merging this into GCJ and I need to 
> know what to test for.

Did GJC use the old implementation of getSystemClassLoader?

Here are some things that were broken:
- It didn't use AccessController.doPrivilege to get the system property
- It used gnu.java.lang.SystemClassLoader which doesn't work
- If a user specified class loader is used it should still delegate to
the system class loader (instead it completely replaced the system class
loader)

New features:
- You get an extension class loader for free
- The system class loader now does the
SecurityManager.checkPackageAccess check

Eventually we should remove gnu/java/lang/SystemClassLoader.java. I'm
not sure if anyone uses it though.

Regards,
Jeroen