RE: Class loader optimization patch

[Jeroen Frijters]

Hi,

Sorry about that, here is the ChangeLog entry:

2003-03-15  Jeroen Frijters  <address@hidden>

        * vm/reference/java/lang/VMClassLoader.java
        (loadClass): Changed documentation to make clear null can
        be returned to signal that the class wasn't found.
        * java/lang/ClassLoader.java
        (loadClass(String,boolean)): Added check for null return
        value from VMClassLoader.loadClass.
        * vm/reference/java/lang/Class.java
        (forName(String,boolean,ClassLoader)): Likewise. 

I looked at NEWS but I don't think the change needs to be mentioned
there.

Regards,
Jeroen

> 
> -----Original Message-----
> From: Mark Wielaard [mailto:address@hidden 
> Sent: Friday, March 14, 2003 20:41
> To: Jeroen Frijters
> Cc: Brian Jones; address@hidden
> 
> Hi,
> 
> On Thu, 2003-03-13 at 19:53, Jeroen Frijters wrote:
> >
> > Since no one objected, could you please apply this patch?
> 
> Applying patches is always easier if there is a ChangeLog 
> entry included (and if it changes the VM interface a little 
> blurb for the NEWS file).
> Hint. Hint.
> 
> Looks good to me and should be transparent to older 
> implementations. I am looking for another VM to test this 
> with but I noticed that both Kissme and SableVM don't 
> actually use the VMClassLoader so keeping backward 
> compatibility isn't that important anyway.
> 
> Cheers,
> 
> Mark
> 
>