This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [RFC] Remove symbol-reloading option
- From: Doug Evans <dje at google dot com>
- To: Eli Zaretskii <eliz at gnu dot org>, Joel Brobecker <brobecker at adacore dot com>
- Cc: gdb-patches at sourceware dot org
- Date: Tue, 13 Mar 2012 09:57:07 -0700
- Subject: Re: [RFC] Remove symbol-reloading option
- References: <20120228213553.187982461CF@ruffy.mtv.corp.google.com> <83ipipcxd1.fsf@gnu.org>
On Wed, Feb 29, 2012 at 10:23 AM, Eli Zaretskii <eliz@gnu.org> wrote:
>> Date: Tue, 28 Feb 2012 13:35:53 -0800 (PST)
>> From: dje@google.com (Doug Evans)
>>
>> The symbol-reloading option is no longer used for anything.
>> IWBN to delete it, but we could first mark it as deprecated, or whatever.
>
> OK for the documentation parts.
>
> Thanks.
On Wed, Feb 29, 2012 at 8:43 AM, Joel Brobecker <brobecker@adacore.com> wrote:
>> The symbol-reloading option is no longer used for anything.
>> IWBN to delete it, but we could first mark it as deprecated, or whatever.
>
> Normally, I would have recommended a 2-step deprecate-then-remove
> approach, but in this case, I tend to agree with you that we can
> remove immediately. I did a little bit of archeology, and found that
> this variable appear to have been unused since at least start of CVS.
> It was appearing once inside symfile.c:free_named_symtabs, but it was
> commented-out so I deleted all that code back in Jan 2010.
>
> So I don't see anyone using this switch today, and it should be fine
> just removing it.
Given that, I will check this in.
Thanks.