Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: "Marc Khouzam" <marc.khouzam@ericsson.com>
To: "Vladimir Prus" <vladimir@codesourcery.com>,
	        <gdb-patches@sources.redhat.com>,
	"Nick Roberts" <nickrob@snap.net.nz>
Subject: RE: [RFA] fix *stopped for CLI commands
Date: Tue, 10 Feb 2009 20:50:00 -0000	[thread overview]
Message-ID: <6D19CA8D71C89C43A057926FE0D4ADAA06E84D64@ecamlmw720.eamcs.ericsson.se> (raw)
In-Reply-To: <200902061045.18508.vladimir@codesourcery.com>

> Vladimir Prus
> Sent: Friday, February 06, 2009 2:45 AM
> To: gdb-patches@sources.redhat.com; Nick Roberts; Marc Khouzam
> Subject: [RFA] fix *stopped for CLI commands
> 
> This patch fixes this by making MI observer print frame 
> again, into MI uiout,
> if necessary.

I've applied the patch and tried it out.  It works as expected
but it 'allowed' me to run into another problem.

When issuing a CLI command that triggers a ^running event,
the token id is not part of the ^running event anymore.
You can see this in the summarized session below.

26-exec-run
[...]
26^running                     <-- token is there for MI command
*running,thread-id="all"
(gdb) 
[...]
(gdb) 
27run
&"run\n"
[...]
^running                       <-- token is not there for CLI command
*running,thread-id="all"
(gdb) 

I never noticed this before because DSF-GDB never uses CLI commands
that would trigger a ^running.  So, to DSF-GDB it looks like the 
CLI command never completed.

Thanks

Marc


  parent reply	other threads:[~2009-02-10 20:50 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-02-06  7:44 Vladimir Prus
2009-02-06 21:11 ` Tom Tromey
2009-02-07  1:04   ` Joel Brobecker
2009-02-12  8:46   ` Vladimir Prus
2009-02-12 20:37     ` Eli Zaretskii
2009-02-14 20:43       ` Vladimir Prus
2009-02-15 18:22         ` Pedro Alves
2009-02-17 19:25           ` Vladimir Prus
2009-02-17 19:29             ` Pedro Alves
2009-02-09 20:36 ` Daniel Jacobowitz
2009-02-10 20:50 ` Marc Khouzam [this message]
2009-02-10 20:58   ` Vladimir Prus
2009-02-11 14:09     ` Marc Khouzam

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=6D19CA8D71C89C43A057926FE0D4ADAA06E84D64@ecamlmw720.eamcs.ericsson.se \
    --to=marc.khouzam@ericsson.com \
    --cc=gdb-patches@sources.redhat.com \
    --cc=nickrob@snap.net.nz \
    --cc=vladimir@codesourcery.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