indent/lua.vim 等 shiftwidth() を使わず &shiftwidth を参照している indent plugin が存在する #1031

Closed
tyru opened this Issue Feb 19, 2017 · 13 comments

Comments

Projects
None yet
3 participants
@tyru
Member

tyru commented Feb 19, 2017

indent/lua.vim 等 shiftwidth() を使わず &shiftwidth を参照している indent plugin が存在する

質問・報告の内容

#714 ならびに vim/vim#578 で直っていたと思われていましたが、まだ参照している indent plugin は沢山存在しました。
以下が修正コミットです。
tyru/vim@a89bc2a

Vimのバージョン

8.0.0339 のツリーから修正しました。

その他

thinca さんの Lingr の発言で気付きました。

うおおお indent/lua.vim が shiftwidth() 使ってない!!!
http://lingr.com/room/vim/archives/2017/02/15#message-23867077

そうか、以前一括で直してましたね。つまり修正漏れか…
http://lingr.com/room/vim/archives/2017/02/17#message-23870876

@tyru

This comment has been minimized.

Show comment
Hide comment
@tyru

tyru Feb 19, 2017

Member

問題はこれをどのように送るか(というか言い方)なんですが、
「以前 https://groups.google.com/forum/#!topic/vim_dev/mRhdDBJh07Y/discussion で直してもらったけど、修正漏れがあったから修正したよ」で送って大丈夫なものでしょうか?

Member

tyru commented Feb 19, 2017

問題はこれをどのように送るか(というか言い方)なんですが、
「以前 https://groups.google.com/forum/#!topic/vim_dev/mRhdDBJh07Y/discussion で直してもらったけど、修正漏れがあったから修正したよ」で送って大丈夫なものでしょうか?

@k-takata

This comment has been minimized.

Show comment
Hide comment
@k-takata

k-takata Feb 19, 2017

Member

他のは let sw = exists('*shiftwidth') ? shiftwidth() : &sw としてあるのでそれに合わせた方がよくないですかね? vim/vim@89bcfda

修正漏れがあったから修正したよ

それでいいと思います。

Member

k-takata commented Feb 19, 2017

他のは let sw = exists('*shiftwidth') ? shiftwidth() : &sw としてあるのでそれに合わせた方がよくないですかね? vim/vim@89bcfda

修正漏れがあったから修正したよ

それでいいと思います。

@tyru

This comment has been minimized.

Show comment
Hide comment
@tyru

tyru Feb 19, 2017

Member

他のは let sw = exists('*shiftwidth') ? shiftwidth() : &sw としてあるのでそれに合わせた方がよくないですかね?

確かに… ただ vim.vim は単純に shiftwidth() に置換してるみたいです。
他にも python.vim や fortran.vim などは以前から shiftwidth() のみを使うようになってるようです。
vim/vim@705ada1#diff-59df7be2982b9c340efe6e4d137ffd68

他の indent plugin は(Bram 以外の修正もあると思いますが)一旦変数に保存して参照するよう修正されてるものもあるようです。
vim/vim@89bcfda

Member

tyru commented Feb 19, 2017

他のは let sw = exists('*shiftwidth') ? shiftwidth() : &sw としてあるのでそれに合わせた方がよくないですかね?

確かに… ただ vim.vim は単純に shiftwidth() に置換してるみたいです。
他にも python.vim や fortran.vim などは以前から shiftwidth() のみを使うようになってるようです。
vim/vim@705ada1#diff-59df7be2982b9c340efe6e4d137ffd68

他の indent plugin は(Bram 以外の修正もあると思いますが)一旦変数に保存して参照するよう修正されてるものもあるようです。
vim/vim@89bcfda

@tyru

This comment has been minimized.

Show comment
Hide comment
@tyru

tyru Feb 19, 2017

Member

vim/vim#578 で Bram 自身が

By now it should be OK to use shiftwidth() directly. It was added in
patch 7.4.694, more than three years ago.

と言っているのを見つけました。

Member

tyru commented Feb 19, 2017

vim/vim#578 で Bram 自身が

By now it should be OK to use shiftwidth() directly. It was added in
patch 7.4.694, more than three years ago.

と言っているのを見つけました。

@tyru

This comment has been minimized.

Show comment
Hide comment
@tyru

This comment has been minimized.

Show comment
Hide comment
@tyru

tyru Feb 19, 2017

Member

こんな返答が来ました。

Wow, that's a lot. Have you pinged the maintainers?

どう返したものか…

Member

tyru commented Feb 19, 2017

こんな返答が来ました。

Wow, that's a lot. Have you pinged the maintainers?

どう返したものか…

@thinca

This comment has been minimized.

Show comment
Hide comment
@thinca

thinca Feb 19, 2017

Member

「メンテナに連絡したか?」なので、「してない」と答えるしかなさそう。

Member

thinca commented Feb 19, 2017

「メンテナに連絡したか?」なので、「してない」と答えるしかなさそう。

@tyru

This comment has been minimized.

Show comment
Hide comment
@tyru

tyru Feb 19, 2017

Member

ですね。
ただどうにかして Bram 自身にパッチを一括で適用してもらえる方向にできないかと…

とりあえず文面考えたので今メール返信しました。

Member

tyru commented Feb 19, 2017

ですね。
ただどうにかして Bram 自身にパッチを一括で適用してもらえる方向にできないかと…

とりあえず文面考えたので今メール返信しました。

@tyru

This comment has been minimized.

Show comment
Hide comment
@tyru

tyru Feb 20, 2017

Member

vim/vim に GH issue 作りました。
vim/vim#1493

あと以前作ったスプレッドシートも更新しました(シート「2nd」を参照してください)。
The List of Vim indent plugin maintainers - Google spreadsheet

Member

tyru commented Feb 20, 2017

vim/vim に GH issue 作りました。
vim/vim#1493

あと以前作ったスプレッドシートも更新しました(シート「2nd」を参照してください)。
The List of Vim indent plugin maintainers - Google spreadsheet

@k-takata

This comment has been minimized.

Show comment
Hide comment
@k-takata

k-takata Jun 14, 2017

Member

直った? vim/vim@3ec574f

Member

k-takata commented Jun 14, 2017

直った? vim/vim@3ec574f

@tyru

This comment has been minimized.

Show comment
Hide comment
@tyru

tyru Jun 14, 2017

Member

帰ったら照らし合わせてみます

Member

tyru commented Jun 14, 2017

帰ったら照らし合わせてみます

@tyru

This comment has been minimized.

Show comment
Hide comment
@tyru

tyru Jun 14, 2017

Member

すみません、今日はきっちり見る気力がないので明日(か明後日)になります…

Member

tyru commented Jun 14, 2017

すみません、今日はきっちり見る気力がないので明日(か明後日)になります…

@tyru

This comment has been minimized.

Show comment
Hide comment
@tyru

tyru Jun 15, 2017

Member

スプレッドシートと照らし合わせるまでもありませんでした。
git grep -w '&sw' runtime/indent/git grep -w '&shiftwidth' runtime/indent/ してオプションを直接使っている箇所がないか調べましたが、ありませんでした。
よってクローズします。

Member

tyru commented Jun 15, 2017

スプレッドシートと照らし合わせるまでもありませんでした。
git grep -w '&sw' runtime/indent/git grep -w '&shiftwidth' runtime/indent/ してオプションを直接使っている箇所がないか調べましたが、ありませんでした。
よってクローズします。

@tyru tyru closed this Jun 15, 2017

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