This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB 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: [PATCH v4 3/9] add target method delegation


>>>>> "Tom" == Tom Tromey <tromey@redhat.com> writes:

Tom> It seems to me that the best approach is to make a target_delegate_*
Tom> function assert only when there is no de_fault for the corresponding
Tom> method.

I mean, when there *is* a de_fault.

The idea is that if there is a de_fault, then the slot cannot be NULL.
So, delegation must never fail.

On the other hand, if there is no de_fault, then the slot can be NULL,
and what exactly to do depends on the method in question.


But looking more closely at the code on the branch, there is an
assertion in those methods returning something other than void.

I'll think about it some more.

Tom


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