From: Nick Roberts <nickrob@snap.net.nz>
To: Vladimir Prus <ghost@cs.msu.su>
Cc: gdb-patches@sources.redhat.com
Subject: Re: linux native async mode support
Date: Sat, 05 Apr 2008 17:20:00 -0000 [thread overview]
Message-ID: <18423.16925.181684.296006@kahikatea.snap.net.nz> (raw)
In-Reply-To: <200804041546.41673.ghost@cs.msu.su>
> > The helper functions that you have written for common sequences are a good
> > idea as they need only be updated in one place but the three here are the
> > only ones of their kind (they output an extra "^done" over MI commands like
> > -exec-next). Not really worth a dedicated helper function do you agree?
>
> Why there's extra "^done"? Presently, each command is supposed to have
> either "^running" or "^done", not both.
There's an extra "^done" because this is a CLI command entered in MI (and
therfore case CLI_COMMAND: of captured_mi_execute_command). There are many
issues here like should we diallow immediate use of CLI commands now and
require explicit use of "-interpreter-exec console"? Then "-interpreter-exec
console next" doesn't emit "(gdb)/n" after "^running" when in asynchronous
mode, so should we remove it from synchronous mode too (as you have suggested)?
But these are all MI issues and this test is meant to just be a mark in the
sand for asynchronous mode, and one that I had lying around. For the moment, I
don't really want to work on it further
> I realize that fixing this requires
> introducing "*running", so that we still know that the target is resumed.
> I have the patch for *running basically done, so I suggest that I submit it,
> and then we'll have just ^done in this example.
I'm not sure what the output will look like after your change but it may
still not be like normal MI output.
> BTW, I'm not even sure that "^done" vs. "^running" + "^done" is so big
> difference that a helper function cannot be introduced.
The comment suggests that mi_gdb_test will not work but I have not revisited
that since writing the test 18 months or so ago. If true, that could also
make helper functions difficult.
--
Nick http://www.inet.net.nz/~nickrob
next prev parent reply other threads:[~2008-04-05 9:11 UTC|newest]
Thread overview: 37+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-03-14 8:11 Pedro Alves
2008-03-14 21:17 ` Daniel Jacobowitz
2008-03-17 16:05 ` Pedro Alves
2008-03-17 22:05 ` Daniel Jacobowitz
2008-03-18 23:27 ` Pedro Alves
2008-03-18 23:58 ` Daniel Jacobowitz
2008-03-21 15:55 ` Daniel Jacobowitz
2008-03-21 17:19 ` Pedro Alves
2008-03-28 14:48 ` Maciej W. Rozycki
2008-03-28 16:07 ` Pedro Alves
2008-03-28 16:13 ` Daniel Jacobowitz
2008-03-28 16:40 ` Pedro Alves
2008-03-18 0:06 ` Nick Roberts
2008-03-18 23:28 ` Pedro Alves
2008-03-19 3:59 ` Nick Roberts
2008-03-19 16:25 ` Luis Machado
2008-03-19 23:19 ` Pedro Alves
2008-03-19 23:26 ` Pedro Alves
2008-03-20 1:58 ` Nick Roberts
2008-03-21 15:47 ` Daniel Jacobowitz
2008-03-21 15:49 ` Daniel Jacobowitz
2008-03-21 23:02 ` Nick Roberts
2008-03-22 1:25 ` Daniel Jacobowitz
2008-03-22 22:06 ` Nick Roberts
2008-04-01 14:00 ` Daniel Jacobowitz
2008-04-01 15:17 ` Vladimir Prus
2008-04-01 20:09 ` Nick Roberts
2008-04-04 12:34 ` Vladimir Prus
2008-04-05 17:20 ` Nick Roberts [this message]
2008-04-05 22:07 ` Vladimir Prus
2008-04-07 0:06 ` Nick Roberts
2008-04-07 2:33 ` Nick Roberts
2008-03-18 2:47 ` Nick Roberts
2008-03-14 23:10 ` Nick Roberts
2008-03-15 1:58 ` Pedro Alves
2008-03-15 3:11 ` Daniel Jacobowitz
2008-03-17 23:41 ` Nick Roberts
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=18423.16925.181684.296006@kahikatea.snap.net.nz \
--to=nickrob@snap.net.nz \
--cc=gdb-patches@sources.redhat.com \
--cc=ghost@cs.msu.su \
/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