Skip to content

原文の変更に追随 (message) #178

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

Merged
merged 2 commits into from
Sep 17, 2017
Merged

原文の変更に追随 (message) #178

merged 2 commits into from
Sep 17, 2017

Conversation

uakms
Copy link
Contributor

@uakms uakms commented Sep 12, 2017

訳の確認をお願いします。

doc/message.jax Outdated
@@ -756,6 +756,13 @@ Vimが一時ファイルを読み込めなかった。特に Windows では、cm

Exコマンドでもユーザー定義コマンドでもないコマンドを実行しようとした。

*E943* >
コマンドテーブルはアップデートされる必要がある。'make cmdidxs' を実行する。
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

この行はエラーメッセージがそのまま書いてある部分なので、翻訳は不要です。

なお、日本語メッセージファイルでは以下のように訳しています。
https://github.com/vim/vim/blob/49a613f84aaa7cccd1d12a2ffc4ebb1956f973aa/src/po/ja.po#L1300-L1303

doc/message.jax Outdated
*E943* >
コマンドテーブルはアップデートされる必要がある。'make cmdidxs' を実行する。

これはソースコードを変更した時と src/ex_cmds.h にコマンドを追加した時にのみ生
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

「これはソースコードを変更して src/ex_cmds.h にコマンドを追加した時にのみ」辺りがよいかと思います。
ソースコードを変更しても、src/ex_cmds.h に変更がなければ、E943 は発生しません。

doc/message.jax Outdated
コマンドテーブルはアップデートされる必要がある。'make cmdidxs' を実行する。

これはソースコードを変更した時と src/ex_cmds.h にコマンドを追加した時にのみ生
じる。このルックアップテーブルは次を実行してアップデートされる必要がある: >
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

日本語メッセージファイルに合わせるなら
「アップデートされる」→「更新する」
でしょうか。受動態にすると不自然な感じがします。

@k-takata k-takata merged commit cdb66a5 into vim-jp:master Sep 17, 2017
@uakms
Copy link
Contributor Author

uakms commented Sep 17, 2017

ありがとうございます!

@uakms uakms deleted the catch-up-message branch September 17, 2017 13:43
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.

2 participants