This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [RFA, doc RFA] New option set use-deprecated-index-sections


> Date: Thu, 19 Jul 2012 09:27:46 -0700
> From: Doug Evans <dje@google.com>
> Cc: Jan Kratochvil <jan.kratochvil@redhat.com>, gdb-patches@sourceware.org
> 
> On Thu, Jul 19, 2012 at 9:08 AM, Eli Zaretskii <eliz@gnu.org> wrote:
> >> Date: Thu, 19 Jul 2012 01:33:03 +0200
> >> From: Jan Kratochvil <jan.kratochvil@redhat.com>
> >> Cc: gdb-patches@sourceware.org, eliz@gnu.org
> >>
> >> On Thu, 19 Jul 2012 00:33:42 +0200, dje@google.com wrote:
> >> > +  use "-iex 'set use-deprecated-index-sections on'".
> >> [...]
> >> > +`-iex \"set use-deprecated-index-sections on\"' to gdb."),
> >>
> >> It is more Eli's decision but I would find it overquoted.
> >
> > How about removing that sentence altogether?  We have no reason to
> > advertise -iex in that particular warning message.
> 
> I'd like to keep the content of the warning.

The text you suggested is:

  	  warning (_("\
  Skipping deprecated .gdb_index section in %s.\n\
  Do \"set use-deprecated-index-sections on\" to use them anyway.\n\
  This can be done from the command line by passing\n\
  `-iex \"set use-deprecated-index-sections on\"' to gdb."),
	     filename);

I'm saying that mentioning the "set" command is enough, no need to
also say that the command can be invoked through -iex, because _any_
GDB command can be invoked like that.

That said, I won't argue.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]