[RFC] JVMTI CLASS_PREPARE event

[Tom Tromey]

>>>>> "Keith" == Keith Seitz <keiths@redhat.com> writes:

Keith> The patch adds the JVMTI event notification for CLASS_PREPARE. This is
Keith> now the third or fourth place I've put this notification (at least),
Keith> and I know this is still not quite right. It's awfully close, but not
Keith> perfect.

Keith> I welcome suggestions, advice, comments, etc.

This looks good to me.

What is it you need to capture here?  Maybe we can suggest a better
spot.  Though, almost all state transitions happen here, so this is
pretty good.

Maybe wrap the condition in a builtin_expect?

Keith> +	  java::lang::Thread *thread = java::lang::Thread::currentThread ();

You could hoist the local variable 'self' outside the new scope and
reuse it.  If you're into micro-optimization and all.

Tom