Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ftplugin/ocaml.vim を改善したい #426

Closed
cohama opened this issue Jun 30, 2013 · 17 comments
Closed

ftplugin/ocaml.vim を改善したい #426

cohama opened this issue Jun 30, 2013 · 17 comments

Comments

@cohama
Copy link

cohama commented Jun 30, 2013

デフォルトの OCaml の ftplugin なのですが、

  • nnoremap すべきところが nmap になっている
  • ftplugin なのにグローバルにマッピングを定義している箇所がある
  • z マークを潰しているところがある
  • オプションの一貫性がない (g:no_ocaml_map を定義しても一部のマッピングしか無効にならないなど)

などです。
修正は自分でできると思いますが、本家にも取り入れてもらいたいです。

@h-east
Copy link
Member

h-east commented Jun 30, 2013

issue登録ありがとうございます。

一番大事なやつ忘れてます↓
「言語名は Ocalm じゃないぜ! OCalm だぜ!」

@koron
Copy link
Member

koron commented Jun 30, 2013

" Maintainer:  David Baelde        <firstname.name@ens-lyon.org>
"              Mike Leary          <leary@nwlink.com>
"              Markus Mottl        <markus.mottl@gmail.com>
"              Pierre Vittet       <pierre-vittet@pvittet.com>
"              Stefano Zacchiroli  <zack@bononia.it>
"              Vincent Aravantinos <firstname.name@imag.fr>
" URL:         http://www.ocaml.info/vim/ftplugin/ocaml.vim

だれがメンテナなのかわからないけど
http://www.ocaml.info/ を見ると Markus Mottl がメインなのかなぁ。
Markus と vim-dev と Bram へ送ってみると良いと思います。

@thinca
Copy link
Member

thinca commented Jun 30, 2013

OCaml、ねw

@cohama
Copy link
Author

cohama commented Jun 30, 2013

OCaml、ねw

はい。Ocaml でも OCalm でもなく OCaml ですね。

Markus と vim-dev と Bram へ送ってみると良いと思います。

う、もしかしなくても英語ですよね。

@koron
Copy link
Member

koron commented Jun 30, 2013

Bram はオランダ語でも大丈夫なはずです。 😹

@h-east
Copy link
Member

h-east commented Jun 30, 2013

よしっ、作戦成功:wink: OCaml

@cohama
Copy link
Author

cohama commented Jun 30, 2013

英語とオランダ語・・・。英語がいいです。とりあえず、他の人のを真似しながら送ってみます。

@koron
Copy link
Member

koron commented Jun 30, 2013

読むだけでメリットが伝わるパッチなら、

私はOCamlのプラグインを進歩させました。
添付したパッチを是非チェックしてください。

くらいを英語で書けば、なんとかなりますよ。気楽にどうぞ。

@cohama
Copy link
Author

cohama commented Jun 30, 2013

おお、ありがとうございます。その作戦で行きます!

@h-east
Copy link
Member

h-east commented Jul 2, 2013

https://twitter.com/c0hama/status/352047023849078785

そういえば Vim で OCaml 書くときに、インサートモードで ASS と打つと面白いことが起こる

ftplugin/ocaml.vim : 66 にこんなものが!

iabbrev <buffer> ASS (assert (0=1) (* XXX *))

これはちょっとキモい設定かも。
ということで、他の言語用ftpluginでも iabbrev が設定されているのか調べてみました。
設定されているのは ocaml.vim 以外だと ada.vim と zimbu.vim の2ファイルでした。

ada.vim は予約後の短縮形を登録している感じ。
ret --> return
proc --> procedure
pack --> package
func --> function

zimbu.vim は予約後を片っ端から大文字に変換するための登録がわんさか(46個)。
(一部抜粋)
iabbr alias GCUpperSpace("alias")
iabbr arg GCUpperDot("arg")

iabbrev にアクロバティックな設定をしているのは ocaml.vim だけっぽい。
なので作者に「このibbrevキモいからやめてくれないか?他言語ではこんなキモい設定やってないよ」とお願いすればやめてくれるかもしれません:grin:

@cohama
Copy link
Author

cohama commented Jul 21, 2013

ちょっと時間がかかってしまいましたが、vim_dev にメールを送りました。

@h-east
Copy link
Member

h-east commented Jul 21, 2013

@cohama 👍
こちらにpatch貼り付けてディスカッションするのもありです。

@cohama
Copy link
Author

cohama commented Jul 22, 2013

おっと、ちょっと先走っちゃいましたかね。

ちなみに、送ったパッチはこんなかんじです。結局 Ocaml は直してません...
https://gist.github.com/6054064

@h-east
Copy link
Member

h-east commented Jul 23, 2013

おっと、ちょっと先走っちゃいましたかね。

No problem。 vim_devに送る前にこちらに修正内容を晒しておけば誰かがtypoやコーディングミスを指摘してくれるかも...くらいの感じです。

vim_devの投稿のリンク貼り付けておきます。
https://groups.google.com/d/msg/vim_dev/2FF0Vyqaw-8/FpGagJmo7ikJ

@k-takata
Copy link
Member

7.4a.047 直前のコミットで取り込まれました。

@cohama
Copy link
Author

cohama commented Jul 31, 2013

おお!
みなさん、いろいろとアドバイスありがとうございました。

@crazymaster
Copy link
Member

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

No branches or pull requests

6 participants