Skip to content

5.9. FFI #78

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 5 commits into from
Feb 13, 2016
Merged

5.9. FFI #78

merged 5 commits into from
Feb 13, 2016

Conversation

ykomatsu
Copy link
Contributor

No description provided.

@ykomatsu ykomatsu changed the title [WIP] 5.9. FFI 5.9. FFI Feb 10, 2016
@ykomatsu
Copy link
Contributor Author

@KeenS 翻訳完了しました。
レビューをお願いいたします

<!-- When declaring the argument types to a foreign function, the Rust compiler can -->
<!-- not check if the declaration is correct, so specifying it correctly is part of -->
<!-- keeping the binding correct at runtime. -->
他言語関数について引数の型を宣言するとき、Rustのコンパイラはその宣言が正しいかどうかを確認することができません。それを正しく指定することは、実行時のバインディングを正しく保つことの一部です。
Copy link
Member

Choose a reason for hiding this comment

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

それを正しく指定することは、実行時のバインディングを正しく保つことの一部です。

原文が読み解きづらいので自信ないのですがやや原文とずれている気がします。
言いたいことは「引数の型のチェックはコンパイラでは出来ないからバインディングを書いた人に全ての責任がある、正しく書けたかは実行時に正しい振舞いをするかでしか分からない。なので(普通のコードを正しく実装するのがコンパイラではなく実装者の責任であるように)バインディングを書く時には引数の型を正しく書くことも求められる」の意味だと思うので

「なのでバインディングが正しく動くためには引数の型を正しく指定してあげる必要があります」
などと意訳してはどうでしょうか。

@KeenS
Copy link
Member

KeenS commented Feb 11, 2016

色々コメントしました。長いかつ英文に癖があってつらかったと思いますがお疲れ様です。
[nits] は些細なので無視して構いません。
確認お願いします。

それと対訳表がコンフリクトしたようなので解決お願いします。

@ykomatsu
Copy link
Contributor Author

@KeenS ありがとうございます。
修正しました。
御確認をお願いいたします

@KeenS
Copy link
Member

KeenS commented Feb 13, 2016

お疲れ様です。マージします。

KeenS added a commit that referenced this pull request Feb 13, 2016
@KeenS KeenS merged commit d819c13 into rust-lang-ja:master Feb 13, 2016
@ykomatsu ykomatsu deleted the ffi branch March 1, 2016 06:37
@tatsuya6502 tatsuya6502 modified the milestone: 1.6 Apr 28, 2016
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.

3 participants