Path: csiph.com!aioe.org!news.glorb.com!usenet.stanford.edu!not-for-mail
From: Eduardo =?utf-8?Q?A=2E_Bustamante_L=C3=B3pez?= <dualbus@gmail.com>
Newsgroups: gnu.bash.bug
Subject: Re: reverse-i-search, multibyte backspace problem
Date: Sat, 18 Jul 2015 20:53:52 -0500
Lines: 60
Approved: bug-bash@gnu.org
Message-ID: <mailman.7133.1437270842.904.bug-bash@gnu.org>
References: <274381437250999@web18h.yandex.ru>
NNTP-Posting-Host: lists.gnu.org
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Cc: bug-bash@gnu.org
To: e est <mtest31@yandex.com>
Envelope-to: bug-bash@gnu.org
Mail-Followup-To: e est <mtest31@yandex.com>, bug-bash@gnu.org
Content-Disposition: inline
In-Reply-To: <274381437250999@web18h.yandex.ru>
User-Agent: Mutt/1.5.23 (2014-03-12)
Precedence: list
Xref: aioe.org gnu.bash.bug:11236

Hello,

Can you please try the 'devel' branch?


There's a fix for this issue already in it:

|  commit 947f04912e4715e7a9df526cd99412bffa729368
|  Author: Chet Ramey <chet.ramey@case.edu>
|  Date:   Tue Jan 27 11:10:49 2015 -0500
|  
|      commit bash-20150116 snapshot


Here's the description of the fix:

|  lib/readline/isearch.c
|         - _rl_isearch_dispatch: if we are in a multibyte locale, make sure to use
|           _rl_find_prev_mbchar when trying to delete characters from the search
|           string, instead of just chopping off the previous byte.  Fixes bug
|           reported by Kyrylo Shpytsya <kshpitsa@gmail.com>


This was reported earlier this year:

  http://lists.gnu.org/archive/html/bug-readline/2015-01/msg00017.html


Or use this to patch:

|  dualbus@yaqui ...src/gnu/bash % git diff origin/master 947f04912e4715e7a9df526cd99412bffa729368 -- lib/readline/isearch.c
|  diff --git a/lib/readline/isearch.c b/lib/readline/isearch.c
|  index 6f6a7a6..d768560 100644
|  --- a/lib/readline/isearch.c
|  +++ b/lib/readline/isearch.c
|  @@ -553,8 +553,16 @@ add_character:
|           do until we have a real isearch-undo. */
|         if (cxt->search_string_index == 0)
|          rl_ding ();
|  -      else
|  +      else if (MB_CUR_MAX == 1 || rl_byte_oriented)
|          cxt->search_string[--cxt->search_string_index] = '\0';
|  +      else
|  +       {
|  +         wstart = _rl_find_prev_mbchar (cxt->search_string, cxt->search_string_index, MB_FIND_NONZERO);
|  +         if (wstart >= 0)
|  +           cxt->search_string[cxt->search_string_index = wstart] = '\0';
|  +         else
|  +           rl_ding ();
|  +       }
|         break;
|   
|       case -4:   /* C-G, abort */

Greetings!

-- 
Eduardo Bustamante
https://dualbus.me/