Skip to content

ownership #53

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 2, 2016
Merged

ownership #53

merged 5 commits into from
Feb 2, 2016

Conversation

ykomatsu
Copy link
Contributor

@ykomatsu ykomatsu commented Feb 1, 2016

No description provided.

@ykomatsu ykomatsu changed the title [WIP] ownership ownership Feb 2, 2016
@ykomatsu
Copy link
Contributor Author

ykomatsu commented Feb 2, 2016

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

@ykomatsu
Copy link
Contributor Author

ykomatsu commented Feb 2, 2016

ごめんなさい。
コードブロック内の翻訳を忘れていたので修正しました。

<!-- we say that we’ve ‘moved’ the thing we refer to. You don’t need some sort of -->
<!-- special annotation here, it’s the default thing that Rust does. -->
「use of moved value」という同じエラーです。
所有権を何か別のものに転送するとき、参照するものを「移転する」と言います。
Copy link
Contributor

Choose a reason for hiding this comment

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

対訳表では move = ムーブ としているので、moved = ムーブした/ムーブ済み などの方が良いかと思いました。(cc: @KeenS )

Copy link
Member

Choose a reason for hiding this comment

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

そうですね。「ムーブする」の方が適切ですね。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@yohhoy @KeenS ありがとうございます。
承知しました。

@KeenS
Copy link
Member

KeenS commented Feb 2, 2016

少しコメントしました。大部分は"move"の訳についてです(校正版のレビュー用のマーカとして全てのものにコメントしました)。確認お願いします。

@ykomatsu
Copy link
Contributor Author

ykomatsu commented Feb 2, 2016

@KeenS 修正しました。
御確認をお願いいたします。

@KeenS
Copy link
Member

KeenS commented Feb 2, 2016

確認しました。お疲れ様です。

KeenS added a commit that referenced this pull request Feb 2, 2016
@KeenS KeenS merged commit 84019d7 into rust-lang-ja:master Feb 2, 2016
@ykomatsu ykomatsu deleted the ownership branch February 2, 2016 20:45
@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.

4 participants