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

translate execute function in eval.jax #94

Merged
merged 7 commits into from Jan 21, 2017
Merged

Conversation

yassu
Copy link
Contributor

@yassu yassu commented Jan 9, 2017

#48

@k-takata
Copy link
Member

k-takata commented Jan 9, 2017

まだ作業中ということでいいですか?

@yassu
Copy link
Contributor Author

yassu commented Jan 9, 2017

あ、すみません. 勘違いしてました.
はい, 作業中です.
WIP つけます.

@yassu yassu changed the title translate execute function in eval.jax [WIP] translate execute function in eval.jax Jan 9, 2017
@yassu yassu changed the title [WIP] translate execute function in eval.jax translate execute function in eval.jax Jan 14, 2017
@yassu
Copy link
Contributor Author

yassu commented Jan 14, 2017

一通り作業終わった気になってます.
何かありましたら, お願いいたします.

`:redir`, error messages are dropped. When using an external
command the screen may be messed up, use `system()` instead.
オプションの {silent} の引数は以下の値を取ることができる:
"" `:silent`を使わない
Copy link
Member

Choose a reason for hiding this comment

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

空白1つ多い気がします。

"silent!" `:silent!` を使う
デフォルトは 'silent' である. "silent!" の Note , `redir`とは
異なり, エラーメッセージは削除される. もし 外部コマンドを使うこ
とがうんざりするなら, 代わりに`system()` を使うことができる.
Copy link
Member

Choose a reason for hiding this comment

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

ここは、「この "silent!" は redir とは異なり、エラーメッセージが削除される事に注意しなさい。」って感じかと。


To get a list of lines use |split()| on the result: >
行のリストを得るために,
Copy link
Member

Choose a reason for hiding this comment

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

, の代わりに を使って下さい。あと行末まで改行せず同じインデントになる様に改行して下さい。


To get a list of lines use |split()| on the result: >
行のリストを得るために,
出力に`split()`を実行することができる: >
Copy link
Member

Choose a reason for hiding this comment

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

原文通り |split()| にして下さい。

*E930*
It is not possible to use `:redir` anywhere in {command}.
{command}の中で いつでも `:redir`を使うことができるわけではない.
Copy link
Member

@mattn mattn Jan 14, 2017

Choose a reason for hiding this comment

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

「{command} の中のどこかで :redir は使う事はできない。」
あと、{command} の後にスペース。

split(execute('args'), "\n")

< When used recursively the output of the recursive call is not
included in the output of the higher level call.
< 再帰的に使用されると、再帰呼び出しの出力は、上位呼び出しの出力
Copy link
Contributor

Choose a reason for hiding this comment

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

行頭の半角スペースは不要です。

@yassu
Copy link
Contributor Author

yassu commented Jan 15, 2017

指摘されていた点は直したと思います.
確認をお願いいたします.

lines are executed one by one.
This is equivalent to: >
Exコマンドもしくはコマンドを実行して、出力を文字列として返す。
{command}は文字列かリストとなることができる。リストの場合はそれ
Copy link
Member

Choose a reason for hiding this comment

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

「となることができる」はちょっと不自然な感じがするので、「が使える」あたりでどうでしょう。

This is equivalent to: >
Exコマンドもしくはコマンドを実行して、出力を文字列として返す。
{command}は文字列かリストとなることができる。リストの場合はそれ
らの行は一行ずつ実行される。これは同値である: >
Copy link
Member

Choose a reason for hiding this comment

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

原文に合わせて、「実行される。」のあとで改行。
「同値である」→「以下と同等である」あたりが自然かと思います。

`:redir`, error messages are dropped. When using an external
command the screen may be messed up, use `system()` instead.
オプションの {silent} の引数は以下の値を取ることができる:
"" `:silent`を使わない
Copy link
Member

@k-takata k-takata Jan 15, 2017

Choose a reason for hiding this comment

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

`:silent` のあとに空白を入れましょう。
クォーテーションの前後には空白をお願いします。(他の箇所も同様)

Copy link
Member

Choose a reason for hiding this comment

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

バッククォートの前後に空白を入れるかどうかは合意がなされていませんが、この行とその下の行で統一されていないので、どちらかに合わせてください。

"" `:silent`を使わない
"silent" `:silent` を使う
"silent!" `:silent!` を使う
デフォルトは 'silent' である。"silent!"の Note "silent!"は
Copy link
Member

Choose a reason for hiding this comment

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

「"silent!"の」は重複しているので削除。
「Note "silent!"は」→「Note "silent!" は」(Noteのあとの空白2つは1個に。)


To get a list of lines use |split()| on the result: >
行のリストを得るために、出力に|split()|を実行することができる:
Copy link
Member

Choose a reason for hiding this comment

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

「実行する」→「使う」
原文は use ですので。

"silent!" `:silent!` を使う
デフォルトは 'silent' である。"silent!"の Note "silent!"は
`redir`とは異なり、エラーメッセージは削除されることに注意するこ
と。もし外部コマンドを使うことがうんざりするなら、代わりに
Copy link
Member

Choose a reason for hiding this comment

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

"the screen may be messed up" の部分が訳されていません。
「外部コマンドを使うと画面がめちゃくちゃになるかもしれないので、」というような感じで。

@yassu
Copy link
Contributor Author

yassu commented Jan 15, 2017

ありがとうございます.
修正を行いました.

@@ -3335,8 +3335,9 @@ executable({expr}) *executable()*

execute({command} [, {silent}]) *execute()*
Exコマンドもしくはコマンドを実行して、出力を文字列として返す。
{command}は文字列かリストとなることができる。リストの場合はそれ
らの行は一行ずつ実行される。これは同値である: >
{command}は文字列かリストを使える。リストの場合はそれ
Copy link
Member

Choose a reason for hiding this comment

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

{command} のあとスペース

`redir`とは異なり、エラーメッセージは削除されることに注意するこ
と。もし外部コマンドを使うことがうんざりするなら、代わりに
`system()`を使うことができる。
と。もし外部コマンドを使うと画面がおかしくなるかもしれないので、
Copy link
Member

Choose a reason for hiding this comment

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

「もし外部コマンドを使って画面がおかしくなる様であれば、」

@koron koron mentioned this pull request Jan 16, 2017
80 tasks
@yassu
Copy link
Contributor Author

yassu commented Jan 18, 2017

修正を行いました

@mattn
Copy link
Member

mattn commented Jan 18, 2017

ありがとうございます。k-takata さんの指摘に対する修正が入ってないかも。

@yassu
Copy link
Contributor Author

yassu commented Jan 21, 2017

バッククォートの修正は行いました.
他の修正は入っていると思います.

@mattn mattn merged commit 8376634 into vim-jp:master Jan 21, 2017
@mattn
Copy link
Member

mattn commented Jan 21, 2017

お疲れさまでした。 👍

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.

None yet

4 participants