[Aegisvm-devel] Re: patch to add debug info to printStackTrace

[Philip F. <pw...@us...>]

Hi Gildas

On Tue, 23 Jul 2002, Gildas Bazin wrote:
> Here is my first contribution :) A patch that adds debug information to 
> printStackTrace().
> Basically it parses the SourceFile attributes and the LineNumberTable 
> attributes of the Code attributes in the class files.
> 
> Could someone review this before applying to the cvs because I'm not too 
> familiar yet with aegisvm's internals and there might be some mistakes.
> 
> Hope you'll find this useful,

Yes, this is definitely useful.  This is going to help debugging a lot.
I've just skimmed through the code a bit, and it looks okay.  So, out of
which version (or CVS tag) is you patch made.  I'll need this in order
to actually integrate your code into the CVS.  I think what I will do
is to add a configure switch so that your new feature is on by default,
and for those who really want to reduce class size when the VM is
deployed, they can choose to turn off the feature.  What do you think?

> PS: there's a also a big patch to jni.c coming :)

I am just curious... what is this big patch going to do?  Can't wait
to see it.

If you are tracking the CVS, you'll notice that I am doing quite a 
bit of restructuring recently.  Would you like to subscribe to
the development mailing list so that you can be kept up to date
about when it is a good time to synchronize your code with the CVS
repository?

Again, thanks a lot for your patch.

Philip

--
Philip W. L. Fong            pw...@us...
The Aegis VM Project         http://aegisvm.sourceforge.net

The Aegis VM Project is an on-going effort to implement a lightweight,
secure JVM.  It will eventually feature a modular architecture, Proof
Linking, that supports pluggable verification modules.