This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: RFA info macro [-at LOCATION,] (v2)
- From: Doug Evans <xdje42 at gmail dot com>
- To: David Taylor <dtaylor at usendtaylorx2l dot lss dot emc dot com>
- Cc: "gdb-patches at sourceware dot org" <gdb-patches at sourceware dot org>, Eli Zaretskii <eliz at gnu dot org>
- Date: Wed, 19 Nov 2014 18:38:36 -0800
- Subject: Re: RFA info macro [-at LOCATION,] (v2)
- Authentication-results: sourceware.org; auth=none
- References: <31913 dot 1415655693 at usendtaylorx2l> <83mw7ymmdw dot fsf at gnu dot org> <8085 dot 1416430826 at usendtaylorx2l>
On Wed, Nov 19, 2014 at 1:00 PM, David Taylor
<dtaylor@usendtaylorx2l.lss.emc.com> wrote:
> This adds a NEWS entry as requested by Eli and I believe addresses his
> concerns with regard to gdb/doc/gdb.texinfo. The other two files
> gdb/macrocmd.c and gdb/testsuite/gdb.base/info-macros.exp are unchanged
> from the previous submission. I updated the gdb/ChangeLog entry to
> include the gdb/NEWS addition..
>
> A few releases ago it was possible to set a location that would be used
> by 'info location' by typing something like:
>
> list file.c:42
>
> and then do:
>
> info macro MACRONAME
>
> . Sometime between release 7.1 and 7.8 that stopped working.
>
> The following patch adds the option [-at LOCATION,] to 'info macro' to
> enable the use of a user selected location as a documented feature.
Hi.
The "," in "-at LOCATION," seems a bit random, relative to other commands.
Maybe it is the best way to go.
If so, I'd like to see the reasons why it exists documented in the code.
Can we just remove the , and require -- when necessary?