Visual Block 挿入でマルチバイト文字が壊れる

[ynkdir]

とりあえず登録だけ…

aaa
あああ
bbb

gg0l<C-V>jjIx<Esc>

axaa
 x <81><82>ああ
bxbb

[mattn]

ここだと思うんですが

https://github.com/vim-jp/vim/blob/master/src/ops.c#L617

どっちにしましょうかね。

axaa
 xあああ
bxaa

axaa
あxああ
bxaa

[mattn]

とりあえず

axaa
xあああ
bxaa

で修正してみました。
※ビジュアルがそう見えたので

diff -r 7b2d05a51e76 src/ops.c
--- a/src/ops.c Wed Jul 23 21:10:44 2014 +0200
+++ b/src/ops.c Mon Jul 28 11:41:18 2014 +0900
@@ -609,6 +609,14 @@
        }
    }

+#ifdef FEAT_MBYTE
+   if (has_mbyte && spaces > 0 && (*mb_ptr2cells)(oldp) > 1)
+   {
+       spaces--;
+       count--;
+   }
+#endif
+
    newp = alloc_check((unsigned)(STRLEN(oldp)) + s_len + count + 1);
    if (newp == NULL)
        continue;

[ichizok]

A だと壊れます

aaa
あああ
bbb

gg0<C-V>jjAx<Esc>

axaa
<e3><81>x<82>ああ
bxbb

[mattn]

@ichizok パッチ済みででしょうか？

[ichizok]

パッチ済みででしょうか？

そうです。I だとOKでした。
7.4.383 / CentOS 6.5

[mattn]

あーなるほど。了解です。

[mattn]

diff -r 7b2d05a51e76 src/ops.c
--- a/src/ops.c Wed Jul 23 21:10:44 2014 +0200
+++ b/src/ops.c Mon Jul 28 16:20:20 2014 +0900
@@ -609,6 +609,25 @@
        }
    }

+#ifdef FEAT_MBYTE
+   if (has_mbyte && spaces > 0)
+   {
+       if (b_insert)
+       {
+       int off = (*mb_head_off)(oldp, oldp + offset + spaces);
+       spaces -= off;
+       count -= off;
+       }
+       else
+       {
+       int off = (*mb_off_next)(oldp, oldp + offset);
+       offset += off;
+       spaces = 0;
+       count = 0;
+       }
+   }
+#endif
+
    newp = alloc_check((unsigned)(STRLEN(oldp)) + s_len + count + 1);
    if (newp == NULL)
        continue;

こんなのでどうでしょうか？

[ichizok]

よさげです。

[mattn]

なんかありそうな気もするのでしばらくランニングします。

[ynkdir]

ありがとうございます。

axaa
xあああ
bxaa

私もこれがいいと思います。

もしかしたら #590 と同じ問題だったかもしれず
そのばあいあとで #590 が直ると二重な感じになるのであれかなと思いました。

[mattn]

https://groups.google.com/forum/#!topic/vim_dev/XsubMBzhRSg

[k-takata]

7.4.408
https://groups.google.com/d/topic/vim_dev/ffBgaCLNfac/discussion