[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Bug-readline] {forward,reverse}-search-history binding bug
From: |
Chet Ramey |
Subject: |
Re: [Bug-readline] {forward,reverse}-search-history binding bug |
Date: |
Fri, 3 Nov 2017 15:33:21 -0400 |
User-agent: |
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.12; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 |
On 10/27/17 12:55 AM, Yuriy Ershov wrote:
> Ah, it seems I understand what is happening!
> InĀ *-search-history mode there is no ANY delay allowed for ESC key.
Not quite. This comment is appropriate:
/* ESC still terminates the search, but if there is pending
input or if input arrives within 0.1 seconds (on systems
with select(2)) it is used as a prefix character
with rl_execute_next. WATCH OUT FOR THIS! This is intended
to allow the arrow keys to be used like ^F and ^B are used
to terminate the search and execute the movement command.
XXX - since _rl_input_available depends on the application-
settable keyboard timeout value, this could alternatively
use _rl_input_queued(100000) */
> It looks like in *-search-history mode readline is "previewing" the next
> input char and prefers to escape back to "normal" mode if there is anything
> unusual.
If there is more input or if input arrives within a tenth of a second.
> This is the way the Up and Down keys work.
> Also, if *-search-history is bound to a key starting with ESC, it detects
> the next ESC key and bails out to the normal mode, leaving that ESC in the
> input queue.
Only if ESC is in the set of characters that terminates the search. If you
remove ESC from the value of the readline `isearch-terminators' variable,
you should have better luck.
> Then readline reads the sequence: ah, there's a *-search-history command,
> let's do the search. But because it has just entered the mode, the buffer
> is empty.
Right.
> Does it look like the truth?
Pretty close. A nice job of investigation.
Chet
--
``The lyf so short, the craft so long to lerne.'' - Chaucer
``Ars longa, vita brevis'' - Hippocrates
Chet Ramey, UTech, CWRU address@hidden http://cnswww.cns.cwru.edu/~chet/
- Re: [Bug-readline] {forward,reverse}-search-history binding bug, Chet Ramey, 2017/11/01
- Re: [Bug-readline] {forward,reverse}-search-history binding bug,
Chet Ramey <=
- Re: [Bug-readline] {forward,reverse}-search-history binding bug, Yuriy Ershov, 2017/11/12
- Re: [Bug-readline] {forward,reverse}-search-history binding bug, G. Branden Robinson, 2017/11/12
- Re: [Bug-readline] {forward,reverse}-search-history binding bug, Yuriy Ershov, 2017/11/12
- Re: [Bug-readline] {forward,reverse}-search-history binding bug, G. Branden Robinson, 2017/11/12
- Re: [Bug-readline] {forward,reverse}-search-history binding bug, Yuriy Ershov, 2017/11/12
- Re: [Bug-readline] {forward,reverse}-search-history binding bug, Chet Ramey, 2017/11/13