From: Andrew Cagney <ac131313@redhat.com>
To: Bob Rossi <bob_rossi@cox.net>
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: Fri, 28 Mar 2003 15:26:00 -0000 [thread overview]
Message-ID: <3E846994.3040708@redhat.com> (raw)
In-Reply-To: <20030321131614.GA14884@white>
> 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
next prev parent reply other threads:[~2003-03-28 15:26 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 [this message]
2003-03-30 4:06 ` Bob Rossi
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 21:59 ` Andrew Cagney
2003-04-03 2:54 ` Eli Zaretskii
2003-04-02 20:24 Bob Rossi
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=3E846994.3040708@redhat.com \
--to=ac131313@redhat.com \
--cc=bob_rossi@cox.net \
--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