Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Bob Rossi <bob_rossi@cox.net>
To: Andrew Cagney <ac131313@redhat.com>
Cc: Jason Molenda <jason-swarelist@molenda.com>,
	gdb-patches@sources.redhat.com
Subject: Re: Adding -file-list-exec-source-file command to GDB/MI
Date: Sun, 30 Mar 2003 04:06:00 -0000	[thread overview]
Message-ID: <20030330040652.GA8455@white> (raw)
In-Reply-To: <3E846994.3040708@redhat.com>

On Fri, Mar 28, 2003 at 10:26:12AM -0500, Andrew Cagney wrote:
> >On Fri, Mar 21, 2003 at 01:55:32AM -0800, Jason Molenda wrote:
> >
> >>Hello Bob,
> >>
> >>My approval isn't needed for these patches or anything, I'm just
> >>an interested observer making comments.
> >>
> >>On Thu, Mar 20, 2003 at 05:44:54PM -0500, Bob Rossi wrote:
> >>
> >
> >>> This change essentially adds the command -file-list-exec-source-file to
> >>> the mi commands. 
> >
> >>
> >>I don't understand why this command is useful.
> >>
> >>A UI can get the filename of the currently-executing source file
> >>easily enough with "stack-list-frames 0 1".  The pathname is returned
> >>as it was recorded in the debug info from the compiler - it might
> >>be an absolute path or it might be a relative path.  
> >
> >
> >At a minumum, it is a strong convienence function for the front end to
> >gdb. It guarentees that the front end is thinking about the same file
> >that gdb is. The front end needs to know about absolute paths. It cares
> >nothing about relative paths.
> >
> >
> >>
> >>If the path is relative, gdb will interpret that pathname based on
> >>the directory gdb was invoked--which presumably the UI did itself.
> >>Or it will be interpreted relative to any paths added with the
> >>"dir" (CLI) / "environment-directory" (MI) command, which the UI
> >>would have added as well.  (or it can get the list of paths with
> >>the environment-directory command without any arguments)
> >>
> >>Why does this information have to be provided by gdb?
> >
> >
> >The best answer probably is, because its been provided for the last
> >decade ( with annotation 1 and 2 ). I strongly believe that just because
> >gdb is switching its interface to front ends, doesn't mean it should
> >take away functionality that was provided before.
> >
> >However, in my opinion, It doesn't really make sense that each front 
> >that implements an interface to gdb figure out how to do each of the 
> >steps provided above.  Especially since gdb is already doing all that 
> >work.
> >
> >Why repeat the functionality in all of the front ends to gdb?
> >
> >It would seem that the best solution would be if this command could be
> >automatically run ( on the front end's request ) every time the source
> >file or line number changed. Just like annotation 1 or 2.
> 
> Sounds like the MI interface should provide both (hey if the client 
> wants a choice of 14 different salad dressings, then who is gdb to argue).
> 
> Someone want to add an extra field, containing that absolute path, to 
> the stack-list-frames command?
> 
> Andrew
> 
> 

Sure, I'll look into that. I would also like to add the extra field to
-break-list. Does that make sense to everyone else?

Bob Rossi


  reply	other threads:[~2003-03-30  4:06 UTC|newest]

Thread overview: 36+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-03-20 22:44 Bob Rossi
2003-03-21  4:34 ` Eli Zaretskii
2003-03-21  9:53 ` Jason Molenda
2003-03-21 13:16   ` Bob Rossi
2003-03-28 15:26     ` Andrew Cagney
2003-03-30  4:06       ` Bob Rossi [this message]
2003-03-31  4:22         ` Andrew Cagney
2003-03-31  8:13           ` Jason Molenda
2003-03-31 14:31             ` Andrew Cagney
2003-03-31 17:16               ` Jason Molenda
2003-03-31 18:52                 ` Andrew Cagney
2003-03-31 19:20                   ` Jason Molenda
2003-03-31 20:12                     ` Andrew Cagney
2003-03-26 22:48 ` Andrew Cagney
2003-03-28 14:35   ` Bob Rossi
2003-03-28 15:30 ` Andrew Cagney
2003-03-28 16:05   ` Bob Rossi
2003-03-28 16:10     ` Daniel Jacobowitz
  -- strict thread matches above, loose matches on Subject: below --
2003-04-02 20:24 Bob Rossi
2003-04-02 20:24 Bob Rossi
2003-04-02 21:59 ` Andrew Cagney
2003-04-03  2:54 ` Eli Zaretskii
2003-03-29  0:41 Bob Rossi
2003-03-31  5:38 ` Andrew Cagney
2003-03-31 18:46   ` Bob Rossi
2003-03-31 18:56     ` Andrew Cagney
2003-04-02 16:15   ` Bob Rossi
2003-04-02 16:27     ` Andrew Cagney
2003-03-10  0:31 Bob Rossi
2003-03-10  4:28 ` Eli Zaretskii
2003-03-18  2:10   ` Bob Rossi
2003-03-18  5:46     ` Eli Zaretskii
2003-02-10 22:28 Michael Elizabeth Chastain
2003-02-10 22:15 Bob Rossi
2003-02-11  6:39 ` Eli Zaretskii
2003-02-13  3:00   ` Bob Rossi

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=20030330040652.GA8455@white \
    --to=bob_rossi@cox.net \
    --cc=ac131313@redhat.com \
    --cc=gdb-patches@sources.redhat.com \
    --cc=jason-swarelist@molenda.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