[PATCH 2/6] Add annex in an async remote notification.
Pedro Alves
palves@redhat.com
Thu Sep 26 18:43:00 GMT 2013
Hi Yao,
On 08/19/2013 02:55 AM, Yao Qi wrote:
> In order to support "Trace:status" notification and other similar
> usage (such as "Point:modified", about a breakpoint is modified), we
> introduce "annex" in the async remote notification, which is helpful
> to give us more details what the contents about in the async remote
> notification. The annex in each notification is optional. In the
> RSP:
Please help me understand this, as it's not obvious to me.
This mechanism adds a bunch of new code. I've read the series and the
descriptions a few times, and I still haven't managed to find where
the rationale behind these annexes is (or figure it out myself). :-(
_Why_ are they necessary? What problem do they solve, that simply
calling these notifications "Trace-status", "Point-modified", and
later other new notifications "Trace-whatever-else", "Point-whatnot",
etc. wouldn't solve? If it's just neat grouping, than it doesn't
look like worthwhile.
The only thing that comes to mind is ordering -- that is, all
event with the same base notification handled on a FIFO basis,
even if they have different annexes. But that doesn't look like
to be the reason -- given that the other annex
hinted -- Point:modified -- belongs to a different notification.
--
Pedro Alves
More information about the Gdb-patches
mailing list