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

:help! で E478 が発生する #49

Closed
thinca opened this issue Sep 27, 2011 · 52 comments
Closed

:help! で E478 が発生する #49

thinca opened this issue Sep 27, 2011 · 52 comments

Comments

@thinca
Copy link
Member

thinca commented Sep 27, 2011

詳細: http://d.hatena.ne.jp/thinca/20100428/1272382745

ドキュメントの記述と実際の動作が異なります。動作を変えるか、ドキュメントを :h[elp]! {subject} にするべき。

@thinca
Copy link
Member Author

thinca commented Sep 27, 2011

動作を変える場合

diff -r 9304e67c02f8 src/ex_cmds.c
--- a/src/ex_cmds.c     Wed Feb 09 11:00:26 2011 +0900
+++ b/src/ex_cmds.c     Tue Sep 27 16:16:44 2011 +0900
@@ -5527,12 +5527,6 @@
        }
        arg = eap->arg;

-       if (eap->forceit && *arg == NUL)
-       {
-           EMSG(_("E478: Don't panic!"));
-           return;
-       }
-
        if (eap->skip)      /* not executing commands */
            return;
     }

ドキュメントを変える場合

diff -r 9304e67c02f8 runtime/doc/helphelp.txt
--- a/runtime/doc/helphelp.txt  Wed Feb 09 11:00:26 2011 +0900
+++ b/runtime/doc/helphelp.txt  Tue Sep 27 16:18:10 2011 +0900
@@ -92,7 +92,7 @@
                                :help so<C-V><CR>only
 <                      {not in Vi}

-:h[elp]! [subject]     Like ":help", but in non-English help files prefer to
+:h[elp]! {subject}     Like ":help", but in non-English help files prefer to
                        find a tag in a file with the same language as the
                        current file.  See |help-translated|.

@ghost ghost assigned thinca Sep 27, 2011
@mattn
Copy link
Member

mattn commented Sep 27, 2011

もしくは現在のバッファがhelpかどうかを見て動作を変えるとか...ってのも考えられるけど、僕的には後者。

@thinca
Copy link
Member Author

thinca commented Sep 27, 2011

Don't panic! は消したくないんですね。わかります。

@thinca
Copy link
Member Author

thinca commented Sep 27, 2011

両方提出して Bram さんに選ばせるって手もありますね。

@thinca
Copy link
Member Author

thinca commented Sep 27, 2011

練習がてら現在バッファがhelpか見るバージョンも作ってみた。

diff -r 9304e67c02f8 src/ex_cmds.c
--- a/src/ex_cmds.c     Wed Feb 09 11:00:26 2011 +0900
+++ b/src/ex_cmds.c     Tue Sep 27 16:54:40 2011 +0900
@@ -5527,7 +5527,7 @@
        }
        arg = eap->arg;

-       if (eap->forceit && *arg == NUL)
+       if (eap->forceit && *arg == NUL && !curbuf->b_help)
        {
            EMSG(_("E478: Don't panic!"));
            return;

@mattn
Copy link
Member

mattn commented Sep 27, 2011

Bramにぐうの音も言わさないつもりか...

@koron
Copy link
Member

koron commented Sep 27, 2011

面白いので両方でやってみてw

@thinca
Copy link
Member Author

thinca commented Sep 27, 2011

了解です。3つ目はどうしますかね? ある意味一番妥当な気もする。

@koron
Copy link
Member

koron commented Sep 27, 2011

私も Don't panic は残して欲しい派なので、2番目と3番目で選んでもらって欲しいですw

@thinca
Copy link
Member Author

thinca commented Sep 27, 2011

了解です。

@mattn
Copy link
Member

mattn commented Oct 17, 2011

@thinca さん、これ出してないみたいですが何か問題あり?

@thinca
Copy link
Member Author

thinca commented Oct 17, 2011

強いて言うなら、私の英語力に問題があります。

@mattn
Copy link
Member

mattn commented Oct 17, 2011

外国人の @ujihisa を召還すればいいの!?

@thinca
Copy link
Member Author

thinca commented Oct 17, 2011

ぼちぼちやろうとは思っていますが、ぶっちゃけいつになるかわかったもんじゃないので、
その方が早いかもしれません!

@ujihisa
Copy link
Member

ujihisa commented Oct 17, 2011

召還されました

@mattn
Copy link
Member

mattn commented Oct 17, 2011

まず、ujihisaさんには次の翻訳をお願いします。

「恥かしがらなくて、いいんだよ」

@thinca
Copy link
Member Author

thinca commented Oct 17, 2011

え、何の話!?

@mattn
Copy link
Member

mattn commented Oct 17, 2011

次にこうです。

俺のパッチを... あっ...

@ujihisa さん。お願いします。

@ujihisa
Copy link
Member

ujihisa commented Oct 17, 2011

Oh... my Excalibur..

@koron
Copy link
Member

koron commented Oct 17, 2011

訳がウマすぎて朝から辛いwww

@thinca
Copy link
Member Author

thinca commented Apr 1, 2012

やっとこさ出しました。
https://groups.google.com/forum/?fromgroups#!topic/vim_dev/kGMRZ1P1qBg
一応全部の案載せてます。
もし議論になったら多分私には収拾付けられんので誰かフォローしてくれると助かります…。(無責任で申し訳ない)

@thinca
Copy link
Member Author

thinca commented Apr 1, 2012

Bram さんから早速返事来てた。最後のが一番いいとのことで概ね予想通りだけど、
これは取り込む意思があるのか、取り込んでくれと改めて言った方がいいのか(最初にちゃんと言っておけば良かった)…。
どちらだと思います?

@ynkdir
Copy link
Member

ynkdir commented Apr 1, 2012

おつです!
取り込んでくれると思います。

@thinca
Copy link
Member Author

thinca commented Apr 1, 2012

ありがとうございます!
慣れてなくて不安だったのでそう言ってもらえると安心します。
問題を勘違いしている方がいるっぽいけどBramさんには伝わってるしちゃんと返す自信ないので悪いけどスルーさせてもらおう…。

@thinca
Copy link
Member Author

thinca commented Apr 5, 2012

@thinca thinca closed this as completed Apr 5, 2012
@mattn
Copy link
Member

mattn commented Apr 5, 2012

おめでとう

@koron
Copy link
Member

koron commented Apr 5, 2012

ヽ(゚∀゚)メ(゚∀゚)メ(゚∀゚)ノ

@thinca
Copy link
Member Author

thinca commented Apr 5, 2012

あざっす!

@koron
Copy link
Member

koron commented Apr 5, 2012

AzaaS!

@koron
Copy link
Member

koron commented Apr 5, 2012

どんだけ気に入ってるんだ>オレ

@h-east
Copy link
Member

h-east commented Apr 5, 2012

おつです。
WiiS! (wireless internet interesting system)
ここはたのしい無線インターネットですね!

@ujihisa
Copy link
Member

ujihisa commented Apr 5, 2012

このスレはまだまだのびる・・・!

@thinca
Copy link
Member Author

thinca commented Apr 5, 2012

次スレは @ujihisa さんが立てると聞いて。

@mattn
Copy link
Member

mattn commented Apr 5, 2012

終わった issue なのにどんどん書きつづけちゃうんだぜ? ワイルドだろぉ?

@koron
Copy link
Member

koron commented Apr 5, 2012

終わったからこそな!

@wozozo
Copy link

wozozo commented Apr 5, 2012

ぼくも混ぜて!

@ujihisa
Copy link
Member

ujihisa commented Apr 5, 2012

終わりがないのが「終わり」・・これがパッチだ!

@h-east
Copy link
Member

h-east commented Apr 5, 2012

もうエエっす!
MoeeS! (Mostly orient enhance enjoying System)
主に東洋では、楽しんでシステムを強化します!

@wozozo
Copy link

wozozo commented Apr 5, 2012

まっつんが面白いこと言ったら終わりそう

@thinca
Copy link
Member Author

thinca commented Apr 5, 2012

明日起きたらどこまでスレが伸びてるか楽しみにしながら寝ます。

@h-east
Copy link
Member

h-east commented Apr 5, 2012

:help!

@ujihisa
Copy link
Member

ujihisa commented Apr 5, 2012

あわわわわてないでたくさい

@h-east
Copy link
Member

h-east commented Apr 5, 2012

:help!

英語:E478: Don't Paciorek! He is Ponce.
日本語:E478: 助っ人外国人選手はパチョレックではありません!元横浜大洋ホエールズのカルロス・ポンセです。

(ちょっと古すぎかw 誰が分かんねんwww)

@ujihisa
Copy link
Member

ujihisa commented Apr 5, 2012

はい

高度すぎてついていけない・・どうしよう・・

@ujihisa
Copy link
Member

ujihisa commented Apr 9, 2012

ping

@thinca
Copy link
Member Author

thinca commented Apr 9, 2012

pong

@mattn
Copy link
Member

mattn commented Apr 9, 2012

:help!!!
E911: Call 911.

@ujihisa
Copy link
Member

ujihisa commented Apr 9, 2012

:help?

@mattn
Copy link
Member

mattn commented Apr 10, 2012

:help?
E911: Could you leave me alone?

@ujihisa
Copy link
Member

ujihisa commented Jun 5, 2012

age

@mattn
Copy link
Member

mattn commented Jun 6, 2012

sage

@koron
Copy link
Member

koron commented Jun 6, 2012

hage

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

7 participants