Path: csiph.com!3.us.feeder.erje.net!feeder.erje.net!news.linkpendium.com!news.linkpendium.com!panix!usenet.stanford.edu!not-for-mail
From: Chet Ramey <chet.ramey@case.edu>
Newsgroups: gnu.bash.bug
Subject: Re: bind -X shows inactive bindings (bindings removed using bind -r)
Date: Fri, 13 Dec 2019 14:25:36 -0500
Lines: 38
Approved: bug-bash@gnu.org
Message-ID: <mailman.823.1576265143.1979.bug-bash@gnu.org>
References: <CAOdNpm8m-8NQVfWiCQX3CKPordxzZShx6K1_OLTbYg77gf5R=g@mail.gmail.com> <CAFLRLk9-QDjyX0v_9OGc364m_wadcma1_q+unWPXhiE0HUq6xw@mail.gmail.com> <029feb2c-77b4-d999-9cdf-f364a8a2f53c@case.edu>
Reply-To: chet.ramey@case.edu
NNTP-Posting-Host: lists.gnu.org
Mime-Version: 1.0
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 7bit
Cc: chet.ramey@case.edu
To: Koichi Murase <myoga.murase@gmail.com>, bug-bash@gnu.org, =?UTF-8?Q?=c5=a0imon_Let?= <simon.a.let@gmail.com>
Envelope-to: bug-bash@gnu.org
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.14; rv:68.0) Gecko/20100101 Thunderbird/68.3.0
In-Reply-To: <CAFLRLk9-QDjyX0v_9OGc364m_wadcma1_q+unWPXhiE0HUq6xw@mail.gmail.com>
Content-Language: en-US
Precedence: list
Xref: csiph.com gnu.bash.bug:15730

On 12/12/19 8:04 AM, Koichi Murase wrote:

> I also would like to see this problem fixed.  In this February, I
> actually posted to this mailing list a patch fixing this problem (with
> another patch):
> 
> https://lists.gnu.org/archive/html/bug-bash/2019-02/msg00038.html
> 
> Although the first patch was applied to the devel branch, the second
> one which fixes this problem seems not to have been reviewed yet.  I
> updated the patch to fit the current devel branch.  I attach the
> updated patch.  Here is the description quoted from the mail:

I declined to apply this patch because, as you observe, it uses private
readline APIs. Bash doesn't do that. Nor did I think it was the right
approach to the problem.

> Note that in the attached patch the functions in Readline library,
> `_rl_get_keyname' and `_rl_untranslate_macro_value', were made public
> and used from Bash codes.  I think it is worth to make them a part of
> public interface of Readline library.

I'm not convinced.

I appreciate the reminder, though. This was still on my list of issues
to look at, so I took a look.

It takes maybe a dozen lines of code, and no modifications to readline,
to solve the problem with `bind -r'. There are a couple of other remaining
issues, but that one was easy.

Chet

-- 
``The lyf so short, the craft so long to lerne.'' - Chaucer
		 ``Ars longa, vita brevis'' - Hippocrates
Chet Ramey, UTech, CWRU    chet@case.edu    http://tiswww.cwru.edu/~chet/