This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Get rid of "set language local"? (was: Re: [PATCH 23/40] Make language_def O(1))
- From: Pedro Alves <palves at redhat dot com>
- To: Keith Seitz <keiths at redhat dot com>, gdb-patches at sourceware dot org
- Date: Thu, 20 Jul 2017 19:11:57 +0100
- Subject: Get rid of "set language local"? (was: Re: [PATCH 23/40] Make language_def O(1))
- Authentication-results: sourceware.org; auth=none
- Authentication-results: ext-mx05.extmail.prod.ext.phx2.redhat.com; dmarc=none (p=none dis=none) header.from=redhat.com
- Authentication-results: ext-mx05.extmail.prod.ext.phx2.redhat.com; spf=pass smtp.mailfrom=palves at redhat dot com
- Dkim-filter: OpenDKIM Filter v2.11.0 mx1.redhat.com 0DC7A461FE
- Dmarc-filter: OpenDMARC Filter v1.3.2 mx1.redhat.com 0DC7A461FE
- References: <1496406158-12663-1-git-send-email-palves@redhat.com> <1496406158-12663-24-git-send-email-palves@redhat.com> <596D4238.7090905@redhat.com> <5ce823d3-1064-f14b-2e79-2ff9efdb28a0@redhat.com>
On 07/20/2017 06:40 PM, Pedro Alves wrote:
> On 07/18/2017 12:03 AM, Keith Seitz wrote:
>> On 06/02/2017 05:22 AM, Pedro Alves wrote:
>>
>>> Note that "local_language_defn" is gone along the way. AFAICT, it's
>>> just a copy of "auto", so the new code simply maps one to the other.
>>> One fewer place to update when we need to change the language
>>> vector...
>>
>> I've searched the manual, and the "local" language is mentioned only in two places, once in the description of "set language" (where it explains that "local" and "auto" are the same) and once when we describe -data-evaluate-expression's --language flag.
>>
>> Honestly, is there any reason to keep it at all given it is a synonym of auto? [I realize that it doesn't cost much to maintain with this patch.] I'm not asking for any changes in this regard, just throwing the possibility out there. TBH, I didn't even know "local" existed.
>
> I didn't know about "local" until I stumbled on it working on this
> series, either. :-)
>
> I'll send a follow up reply with a different subject
> to see if anyone else has comments on that.
Does anyone here ever use "set language local" instead
of "set language auto"? Do we need to keep "set language local" ?
Seems odd to have two ways to do the exact same thing.
Thanks,
Pedro Alves