Skip to content

4.36 Unsafe #87

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 6 commits into from
Feb 15, 2016
Merged

4.36 Unsafe #87

merged 6 commits into from
Feb 15, 2016

Conversation

mandel59
Copy link
Contributor

No description provided.

@mandel59
Copy link
Contributor Author

ひととおり訳し終えたので、レビューをお願いします。

* `enum` の定義に含まれていない判別子
* `char` における、サロゲートか `char::MAX` を超えた値
* `str` における、UTF-8でないバイト列
* 他言語からRustへのアンワインディングや、Rustから他言語へのアンワインディング
Copy link
Member

Choose a reason for hiding this comment

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

[IMO]
「アンワインド」は分かりにくい気がするのですがどうでしょうか。普通に「巻き戻し」の方が通じそうな気がします。

@KeenS
Copy link
Member

KeenS commented Feb 14, 2016

@mandel59
1点だけコメンントしました。
それと、全体的にコメントアウトした原文の直下(空行を空けない)規約になっているので空けている空行を削除お願いします。

また、管理上レビューを受ける時には[WIP] を外して頂くと非常に助かります。

@mandel59 mandel59 changed the title [WIP] 4.36 Unsafe 4.36 Unsafe Feb 14, 2016
@mandel59
Copy link
Contributor Author

修正しました。

@KeenS
Copy link
Member

KeenS commented Feb 15, 2016

確認しました。ありがとうございます。マージします。

KeenS added a commit that referenced this pull request Feb 15, 2016
@KeenS KeenS merged commit 5d79350 into rust-lang-ja:master Feb 15, 2016
@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