Mirror of the gdb mailing list
 help / color / mirror / Atom feed
From: Andrew Cagney <ac131313@cygnus.com>
To: Keith Seitz <keiths@redhat.com>
Cc: gdb@sources.redhat.com
Subject: Re: [MI] -break-insert: (a)synchronous?
Date: Fri, 14 Jun 2002 09:33:00 -0000	[thread overview]
Message-ID: <3D0A1AE0.4030209@cygnus.com> (raw)
In-Reply-To: <Pine.GSO.4.33.0206131739190.19373-100000@makita.cygnus.com>

> On Thu, 13 Jun 2002, Andrew Cagney wrote:
> 
> 
>> The command was implemented that way to match its documented spec.  I
>> remember wondering about alternate implementations at the time.
>>
>> Sounds like it is time to either define a new command (not capture the
>> events) or change the spec.
> 
> 
> Ok, then I would like to propose that we change the spec to use events and
> only events when a command results in some event. This reduces the
> redundancy.

This means another MI version bump (currently it is on 1).  See how mi0 
vs mi1 was handled (especially in the testsuite).

BTW, there is another necessary interface change lurking in the wings - 
frames need to be changed to lists (dump of frame_id) instead of single 
values - so the version bump is likely a good idea.

How to do it?

I guess prototype the changes with -i=mi2 and then when ready do the 
testsuite suffle for the test cases that are effected.

> I'm working on modifying gdbmi.texinfo, but I am a little confused about
> all the stream options:
> 
> "*" = "exec-async-output"
> "+" = "status-async-output"
> "=" = "notify-async-output"

I don't think there was much checking for compliance of this.

>>From the manual ([] = my comments):
>   o status-async-output contains on-going status information about the
>     progess of slow operation. All status output is prefixed by "+".
>     [I presume that the biggest client for this is downloading to a target.]
>   o exec-async-output contains asynchronous state change on the target
>     (stopped, started, disappeared). All Async output is prefixed by "*".
>     [This seems to be called only by async target state changes.]
>   o notify-async-output contains supplementary information that the client
>     should handle (e.g., a new breakpoint information). All notify
>     output is prefixed by "=".
>     [I don't know where this is used. I am unable to find any references
>      to this.]
> 
>>From the descriptions above, it sounds like event notifications should
> occur on the notify-async-output channel, "=".
> 
> However, if one reads down to the "Command With Side Effects" section, one
> sees:
> 
> -> -symbol-file xyz.exe
> <- *breakpoint,nr="3",address="0x123",source="a.c:123"
> <- (gdb)

I think this is a straight error.

> So, here's how I interpret what should happen:
> 
> (gdb)
> -break-insert main
> =breakpoint-create,number="1"
> ^done,bkpt=...
> (gdb)
> 
> (I will also be proposing that we whack the result in this case, since we
> will get an event notification, too.)

Andrew





  reply	other threads:[~2002-06-14 16:33 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2002-06-13 13:55 Keith Seitz
2002-06-13 17:30 ` Andrew Cagney
2002-06-13 19:27   ` Keith Seitz
2002-06-14  9:33     ` Andrew Cagney [this message]
2002-06-14 10:49       ` Keith Seitz
2002-06-14 11:23         ` Daniel Jacobowitz
2002-06-14 11:48           ` Andrew Cagney

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=3D0A1AE0.4030209@cygnus.com \
    --to=ac131313@cygnus.com \
    --cc=gdb@sources.redhat.com \
    --cc=keiths@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox