Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Nick Roberts <nickrob@snap.net.nz>
To: Daniel Jacobowitz <drow@false.org>
Cc: gdb-patches@sources.redhat.com
Subject: Re: [PATCH: gdb/mi + doco] -var-update
Date: Fri, 15 Jul 2005 22:37:00 -0000	[thread overview]
Message-ID: <17112.15129.400036.830436@farnswood.snap.net.nz> (raw)
In-Reply-To: <20050715152840.GA20960@nevyn.them.org>

Daniel Jacobowitz writes:
 > On Sat, Jul 16, 2005 at 03:12:00AM +1200, Nick Roberts wrote:
 > >  > BTW, if it's OK with you, I would prefer that you add new testcases
 > >  > rather than modifying existing ones.  Yes, the ones that are there are
 > >  > slightly redundant.  But changing what a test case is testing is bad
 > >  > form for long-term results analysis.
 > > 
 > > OK.  I'm not very comfortable with expect, but I'll give it a go.
 > 
 > I have learned a frightening (to me anyway) amount about expect and TCL
 > lately; bug me with any questions you may have.

Thanks.

 > >  > Now that this patch is in, I believe your current gdb-mi.el will work
 > >  > with CVS GDB.  Is that right?  If so, could you post it?  I promise to
 > >  > look at it promptly.
 > > 
 > > Yes it does work, but not perfectly (if it did it could go straight into
 > > the Emacs CVS repository).  It has changed quite a lot since last time.
 > > Unfortunately it still requires Emacs in CVS, but a release is planned.
 > 
 > Confused - I thought we'd decided it belonged with GDB, not with
 > emacs, because of the GDB version dependencies?

The eventual aim is to move it to Emacs CVS.  While MI and gdb-mi.el are
changing , GDB CVS seems a good place for it.  You're right though, since
it currently requires MI features that are only in GDB CVS, it wouldn't
make sense to move it to Emacs CVS yet.

 > I won't try to second-guess you about this code - my LISP sucks.  But
 > could you explain briefly why you changed from -break-insert to "break"
 > in some places?  Yet at the same time changed from until to
 > -exec-until...

I changed "until" to -exec-until as it's an MI command.  I changed
-break-insert back to "break" because the output from -break-insert spills
into the GUD buffer.  Clearly the proper solution is to handle the output
of -break-insert, but I haven't got that far yet.  I don't know how
important that is because gdb-mi.el will always have to be able to handle
CLI user input from the GUD buffer anyway.

 > Anyway, go ahead and check this in.

Committed.  Thanks.

Nick


  reply	other threads:[~2005-07-15 22:37 UTC|newest]

Thread overview: 45+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-02-20  2:06 Nick Roberts
2005-02-20  5:02 ` Eli Zaretskii
2005-02-20  5:51   ` Nick Roberts
2005-02-20 15:31     ` Eli Zaretskii
2005-02-21  4:33       ` Nick Roberts
2005-02-21  7:17         ` Eli Zaretskii
2005-02-22  9:23           ` Nick Roberts
2005-02-22  9:34             ` Eli Zaretskii
2005-02-27  5:03               ` Nick Roberts
2005-02-27 16:53                 ` Eli Zaretskii
2005-02-27 16:56                   ` Nick Roberts
2005-02-28 12:27                     ` Eli Zaretskii
2005-03-19  3:55                 ` Nick Roberts
2005-04-01  1:51                   ` Bob Rossi
2005-04-01 11:01                     ` Nick Roberts
2005-05-02  2:06                     ` Nick Roberts
2005-05-02  4:05                       ` Daniel Jacobowitz
2005-05-02  7:24                         ` Nick Roberts
2005-06-17  3:43                           ` Daniel Jacobowitz
2005-06-17 10:09                             ` Eli Zaretskii
2005-06-17 14:04                               ` Daniel Jacobowitz
2005-06-18  8:53                                 ` Eli Zaretskii
2005-07-03 19:56                                   ` Daniel Jacobowitz
2005-07-04  3:07                                     ` Nick Roberts
2005-07-04  3:51                                       ` Daniel Jacobowitz
2005-07-04  4:55                                     ` Eli Zaretskii
2005-07-04  5:02                                       ` Daniel Jacobowitz
2005-07-04 10:17                                         ` Nick Roberts
2005-07-06 10:00                                         ` Nick Roberts
2005-07-15  1:44                                           ` Daniel Jacobowitz
2005-07-15  3:59                                             ` Nick Roberts
2005-07-15  4:16                                               ` Daniel Jacobowitz
2005-07-15 15:11                                                 ` Nick Roberts
2005-07-15 15:28                                                   ` Daniel Jacobowitz
2005-07-15 22:37                                                     ` Nick Roberts [this message]
2005-07-04 21:15                                       ` Nick Roberts
2005-07-04 22:24                                         ` Eli Zaretskii
2005-07-05  3:25                                           ` Nick Roberts
2005-07-05 19:37                                             ` Eli Zaretskii
2005-07-15  9:42                                               ` Nick Roberts
2005-06-17 11:42                             ` Nick Roberts
2005-06-17 14:06                               ` Daniel Jacobowitz
2005-06-17 23:12                                 ` Nick Roberts
2005-02-21  2:36 ` Andrew Cagney
2005-02-21  3:28   ` 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=17112.15129.400036.830436@farnswood.snap.net.nz \
    --to=nickrob@snap.net.nz \
    --cc=drow@false.org \
    --cc=gdb-patches@sources.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