Re: Serious regression in staging

On Tue, Jan 18, 2005 at 11:47:31PM -0500, Grzegorz B. Prokopski wrote:
> 
> David, I liked your patch for reflection.  I am not a fan of skipping
> some semi-defined frames in VMStackWalker, so if it works and we don't
> have to have this special reflection handling there - that's great.
> 

Hi,

Actually for the bug I was encountering the class context obtained is
useless as it is not the calling class loader to use in that case.

But I agree that all internal Class.forName class loading code should
specify the class loader explicitely.  The way that
Class.forName(String) is implemented assumes that the Class.forName is
called from "user" code and not internally in the class libs.

> You said it fixed problem for SableJIT, but could you please try to
> run any AWT/Swing app and check whether they work (might be in
> interpreter mode, should not make any difference)?
> 
> And... I am curious - but did you try to run Eclipse on SableJIT? :-)
> 

Which one breaks?  All of them?  I will try some GUI apps.

No, I didn't try Eclipse on SableJIT, I will try it some time :)


David

---

David Bélanger
Web page:   http://www.cs.mcgill.ca/~dbelan2/
Public key: http://www.cs.mcgill.ca/~dbelan2/public_key.txt