Re: ksh vi mode: fix region deletion and yanking

2016-10-17 Thread Ingo Schwarze
Hi Theo, Theo Buehler wrote on Fri, Oct 14, 2016 at 09:09:34PM +0200: > The patch reads fine to me and behaves well in some testing. > It is ok tb@ as is. Committed, thanks for checking. > I wonder if for the sake of consistency it would be worth merging the > 'b' and 'B' cases as well as the

Re: ksh vi mode: fix region deletion and yanking

2016-10-14 Thread Theo Buehler
> Most of the word movement functions required complete rewrites, > but the new versions aren't harder to read than the old ones. > > I have done some testing, but a bit more might be in order. > > OK? The patch reads fine to me and behaves well in some testing. It is ok tb@ as is. I wonder if

Re: ksh vi mode: fix region deletion and yanking

2016-10-14 Thread Ingo Schwarze
Hi Dmitrij, Dmitrij D. Czarkoff wrote on Fri, Oct 14, 2016 at 01:11:16PM +0200: > I've noticed that in ksh's vi mode ranged operations are performed > without respect to cursor's position within utf8 byte sequence. Eg.: > > 1. type "echo ั‚ะตัั‚ | hexdump -C" > 2. leave inseart mode > 3. "0",