This is the mail archive of the java-patches@gcc.gnu.org mailing list for the Java project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [RFA/JVMTI] SetEventNotificationMode and SetEventCallbacks


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

Sorry it took me so long to get back to this.
Please feel free to ping me if my delays are blocking you.

Keith> Hmm. Yeah, I suppose I could do something like that. Although I've
Keith> tested this quite a bit, I'm not entirely sure the new cast in this
Keith> function (check_enabled_event) is legal. Alas, gcc doesn't give any
Keith> warnings on this. Could you please scrutinize this?

Keith> +      void (**callback)()
Keith> +	= reinterpret_cast<void (**)()> (&e->env->callbacks) + index;

This looks invalid to me though I don't have a reference to the
standard for it.  Could we use a pointer-to-member instead?
Sorry to keep going around on this.

Tom


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]