Skip to content

Conversation

@thinca
Copy link
Member

@thinca thinca commented Mar 23, 2016

#190

特に反応なければ早いうちにマージしちゃいますが、もし良ければ一応誰か見てもらえると。

@koron
Copy link
Member

koron commented Mar 23, 2016

あ、やっぱ1577までは問題なかった、ってことで良いのでしょうか?

@thinca
Copy link
Member Author

thinca commented Mar 23, 2016

そうですね。Partial 自体が入ったのは 7.4.1559 なんですが、問題が起きたのは 7.4.1577 からになります。

@koron
Copy link
Member

koron commented Mar 23, 2016

ではおKです。

@crazymaster
Copy link
Member

7.4.1639 でも E724 が起きる問題(vital.vimのテスト, incsearch.vim)が直っていないような...

@crazymaster crazymaster changed the title Improve a post about patch 1577 Improve a post about patch 7.4.1577 Mar 23, 2016
@k-takata
Copy link
Member

vim-jp/issues#169 が顕在化してしまいましたか。

@h-east
Copy link
Member

h-east commented Mar 23, 2016

@k-takata 最新Vimでネスト上限を100から1000にして試したけど起きたので、おそらくpartial関連にバグがあるのかも。

@mattn
Copy link
Member

mattn commented Mar 23, 2016

@crazymaster 詳細プリーズ

@k-takata
Copy link
Member

バグというか、partialとdictが循環参照になってしまっているので、永久に表示し続けようとして上限に達しているだけかと。

@mattn
Copy link
Member

mattn commented Mar 23, 2016

はてブは変更済み Patch 1577Patch 7.4.1577
http://b.hatena.ne.jp/entry/vim-jp.org/blog/2016/03/23/take-care-of-patch-1577.html

@mattn
Copy link
Member

mattn commented Mar 23, 2016

例外が出て当然ですね。shortmess+=T してるなら別ですが。

@k-takata
Copy link
Member

それはそれとして、まずはマージしてしまっていいのでは。

@mattn mattn merged commit 32b4b90 into master Mar 23, 2016
@mattn mattn deleted the improve-post-1577 branch March 23, 2016 10:07
@crazymaster
Copy link
Member

以下が該当issueです
thinca/vim-themis#33
haya14busa/incsearch.vim#111

@mattn
Copy link
Member

mattn commented Mar 23, 2016

あざます

@thinca
Copy link
Member Author

thinca commented Mar 23, 2016

マージあざます。

E724 についてはかなり昔に報告があります。今回その対象に partial が追加された感じになりますね。
vim-jp/issues#169

@mattn
Copy link
Member

mattn commented Mar 23, 2016

↑のvitalの件、再現手順が分かるならissueにあげてもらえると助かります。(他力本願

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants