Skip to content

5.7. Error Handling (part 3) #95

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 7 commits into from
Feb 22, 2016

Conversation

tatsuya6502
Copy link
Member

@KeenS @ykomatsu @kw-udon

「5.7. エラーハンドリング」のパート3です。レビューをお願いします。

@tatsuya6502
Copy link
Member Author

@KeenS @ykomatsu @kw-udon

この「5.7. エラーハンドリング」のパート3「標準ライブラリのトレイトによるエラー処理」ですが、そろそろパート4のPRを上げたいので、現状の、レビューが無いままでマージしますね。

この章は、次のパート4で終わりです。できれば、パート4のPRのレビュー時に、パート3部分も通してレビューしていただけると、非常に助かります。

tatsuya6502 added a commit that referenced this pull request Feb 22, 2016
@tatsuya6502 tatsuya6502 merged commit 2492744 into rust-lang-ja:master Feb 22, 2016
<!-- following things: -->
このトレイトは極めて一般的です。
なぜなら、エラーを表す *全て* の型で実装されることを目的としているからです。
この後すぐ見るように、このことが合成可能なコードを書くのに間違いなく役立ちます。
Copy link
Contributor

Choose a reason for hiding this comment

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

[nits]
「このこと"は"合成可能なコードを書くのに」 としたほうが、自然に感じます。

@keiichiw
Copy link
Contributor

既にマージされていますが、レビュー致しました。
ご確認のほど、よろしくお願いします。

tatsuya6502 added a commit to tatsuya6502/the-rust-programming-language-ja that referenced this pull request Feb 25, 2016
tatsuya6502 added a commit to tatsuya6502/the-rust-programming-language-ja that referenced this pull request Feb 25, 2016
@tatsuya6502
Copy link
Member Author

@kw-udon ありがとうございました。修正はこのPR用のブランチと、Part 4 の PR #101 用のブランチと両方にコミットしました。(最初に PR #101 の方にコミットして、こちらのブランチには git cherry-pick で反映)

こちらのブランチにも反映することで、レビューコメントのついた diff が、outdated になることを期待したのですが、マージ済みだと、そうならないんですね。(賢いですねー)

どちらのコミットも同じ内容ですが、マージされるのは PR #101 側のコミットになります。修正箇所は、そちらの diff でご確認ください:

@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.

2 participants