Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Eli Zaretskii via Gdb-patches <gdb-patches@sourceware.org>
To: Pedro Alves <pedro@palves.net>
Cc: gdb-patches@sourceware.org
Subject: Re: [PATCH v4] gdb/manual: Introduce location specs
Date: Fri, 27 May 2022 22:00:27 +0300	[thread overview]
Message-ID: <83mtf2n6lw.fsf@gnu.org> (raw)
In-Reply-To: <d68edc0d-6e0c-a132-2442-21c8dae92e39@palves.net> (message from Pedro Alves on Fri, 27 May 2022 19:50:58 +0100)

> Date: Fri, 27 May 2022 19:50:58 +0100
> From: Pedro Alves <pedro@palves.net>
> Cc: gdb-patches@sourceware.org
> 
> BTW, my text says this:
> 
>  +The location spec may be incomplete, and @value{GDBN} will do its best
>  +to find all the locations in the program that match it.  For example,
>  +a location spec may just indicate a line number and a source filename
>                                                         ^^^^^^^^^^^^^^^
>  +with no directory components, 
>   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
> 
> and this example too:
> 
>  +@item
>  +The location spec specifies a file name, and multiple files in the
>  +program share the same name.

That's about location specs; I was talking about the code location,
where the missing attributes need to be resolved.

> IMO, from this it should be obvious that the code location has directory
> components that you can match.

I don't think I follow: match with what and in what way?  Are you
talking about "matching" of an absolute file name with relative file
names?

> The explicit locations section already says for example:
> 
>  @table @code
>  @item -source @var{filename}
>  The value specifies the source file name.  To differentiate between
>  files with the same base name, prepend as many directories as is necessary
>  to uniquely identify the desired file, e.g., @file{foo/bar/baz.c}.  Otherwise
>  @value{GDBN} will use the first file it finds with the given base
>  name.   This option requires the use of either @code{-function} or @code{-line}.
> 
> I'm not sure we want to go into details like that in the intro section.  Users won't
> be able to use any location spec format without reading the corresponding section,
> so they'll see this.

What I was talking about was not about using the location specs, it
was about explaining what it is and how it differs from the code
location.  I think we have to explain the terminology before we use
it, since it is not entirely trivial.

  reply	other threads:[~2022-05-27 19:01 UTC|newest]

Thread overview: 43+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-05-26 19:42 Pedro Alves
2022-05-27 14:16 ` Eli Zaretskii via Gdb-patches
2022-05-27 15:04   ` Pedro Alves
2022-05-27 15:52     ` Eli Zaretskii via Gdb-patches
2022-05-27 17:11       ` Pedro Alves
2022-05-27 17:31         ` Eli Zaretskii via Gdb-patches
2022-05-27 17:51           ` Pedro Alves
2022-05-27 18:23             ` Eli Zaretskii via Gdb-patches
2022-05-27 18:42               ` Pedro Alves
2022-05-27 18:50                 ` Pedro Alves
2022-05-27 19:00                   ` Eli Zaretskii via Gdb-patches [this message]
2022-05-27 19:30                     ` Pedro Alves
2022-05-28  7:43                       ` Eli Zaretskii via Gdb-patches
2022-05-30 14:38                         ` Pedro Alves
2022-05-27 18:55                 ` Eli Zaretskii via Gdb-patches
2022-05-27 19:05                   ` Pedro Alves
2022-05-27 19:14                     ` Eli Zaretskii via Gdb-patches
2022-05-27 19:17                       ` Pedro Alves
2022-05-27 19:34                         ` Eli Zaretskii via Gdb-patches
2022-05-27 19:38                           ` Pedro Alves
2022-05-28  7:39 ` Eli Zaretskii via Gdb-patches
2022-05-30 14:44   ` [pushed v5] " Pedro Alves
2022-05-30 16:15     ` Eli Zaretskii via Gdb-patches
2022-05-31 11:05       ` [PATCH] Improve clear command's documentation Pedro Alves
2022-05-31 12:36         ` Eli Zaretskii via Gdb-patches
2022-05-31 13:04           ` Pedro Alves
2022-05-31 13:42             ` Eli Zaretskii via Gdb-patches
2022-05-31 14:47               ` Pedro Alves
2022-05-31 16:06                 ` Eli Zaretskii via Gdb-patches
2022-05-31 11:13       ` [PATCH] Explicitly mention yet-unloaded shared libraries in location spec examples Pedro Alves
2022-05-31 11:47         ` Eli Zaretskii via Gdb-patches
2022-05-31 11:17       ` [pushed v5] gdb/manual: Introduce location specs Pedro Alves
2022-05-31 11:31       ` [PATCH] Improve break-range's documentation Pedro Alves
2022-05-31 11:55         ` Eli Zaretskii via Gdb-patches
2022-05-31 12:03           ` Pedro Alves
2022-05-31 12:09             ` Eli Zaretskii via Gdb-patches
2022-06-01 17:17     ` RTe: Location Specs (Was: [pushed v5] gdb/manual: Introduce location specs) Eli Zaretskii via Gdb-patches
2022-06-02 11:10       ` Pedro Alves
2022-06-02 11:49         ` Eli Zaretskii via Gdb-patches
2022-06-02 12:40           ` Pedro Alves
2022-06-02 12:56             ` Eli Zaretskii via Gdb-patches
2022-06-02 13:44               ` Pedro Alves
2022-06-02 16:37                 ` Eli Zaretskii via Gdb-patches

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=83mtf2n6lw.fsf@gnu.org \
    --to=gdb-patches@sourceware.org \
    --cc=eliz@gnu.org \
    --cc=pedro@palves.net \
    /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