RE: [cp-patches] Re: Some VMStackWalker cleanups/simplifications

[Jeroen Frijters]

Robert Lougher wrote:
> Without casting stones, I would like to point out that JamVM 
> and all the other VMs have endured quite a few changes to the
> VM interface recently that seem to have been for IKVMs benefit.

I'm sorry you feel this way, but you can hardly apply this to this
particular issue, Archie designed and implemented the VMStackWalker
interface. The only major change that was mostly for IKVM's benefit was
the rearranging of the system properties (which I think benefits other
VMs too).

> The change to getClassContext() when it was moved from 
> VMSecurityManager to VMStackWalker.  The old version was
> responsible for popping off TWO frames (in addition to those
> for reflection).  The one for VMSecurityManager AND the one 
> for SecurityManager itself.  The new one only pops off the 
> one for itself, i.e. VMStackWalker.

If you have a problem with getClassContext(), feel free to add a count
(or even a class) parameter to that, I wouldn't mind that at all. As
long as getCallingClass[Loader] doesn't change, I'm happy.

Regards,
Jeroen