Path: csiph.com!aioe.org!news.glorb.com!usenet.stanford.edu!not-for-mail
From: e est <mtest31@yandex.com>
Newsgroups: gnu.bash.bug
Subject: Re: reverse-i-search, multibyte backspace problem
Date: Sun, 19 Jul 2015 04:25:16 +0200
Lines: 70
Approved: bug-bash@gnu.org
Message-ID: <mailman.7134.1437272726.904.bug-bash@gnu.org>
References: <274381437250999@web18h.yandex.ru> <20150719015352.GA32433@ma.sdf.org>
NNTP-Posting-Host: lists.gnu.org
Mime-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable
Cc: "bug-bash@gnu.org" <bug-bash@gnu.org>
To: =?utf-8?B?RWR1YXJkbyBBLiBCdXN0YW1hbnRlIEzDs3Bleg==?= <dualbus@gmail.com>
Envelope-to: bug-bash@gnu.org
In-Reply-To: <20150719015352.GA32433@ma.sdf.org>
Precedence: list
Xref: aioe.org gnu.bash.bug:11237

Hello,

Thanks for pointing out the fix. I've tried the "devel" branch, and could=
n't reproduce the bug there.

Sorry for the disturbance, I should have checked whether the master branc=
h really represents the bleeding edge of development.

19.07.2015, 03:53, "Eduardo A. Bustamante L=C3=B3pez" <dualbus@gmail.com>=
:
> 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:
>
> =C2=A0=C2=A0http://lists.gnu.org/archive/html/bug-readline/2015-01/msg0=
0017.html
>
> Or use this to patch:
>
> | dualbus@yaqui ...src/gnu/bash % git diff origin/master 947f04912e4715=
e7a9df526cd99412bffa729368 -- 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 =3D=3D 0)
> | rl_ding ();
> | - else
> | + else if (MB_CUR_MAX =3D=3D 1 || rl_byte_oriented)
> | cxt->search_string[--cxt->search_string_index] =3D '\0';
> | + else
> | + {
> | + wstart =3D _rl_find_prev_mbchar (cxt->search_string, cxt->search_st=
ring_index, MB_FIND_NONZERO);
> | + if (wstart >=3D 0)
> | + cxt->search_string[cxt->search_string_index =3D wstart] =3D '\0';
> | + else
> | + rl_ding ();
> | + }
> | break;
> |
> | case -4: /* C-G, abort */
>
> Greetings!
>
> --
> Eduardo Bustamante
> https://dualbus.me/