リファレンス追加: std::vec::IntoIter および vecモジュールの誤字修正#16
リファレンス追加: std::vec::IntoIter および vecモジュールの誤字修正#16okakatsuo wants to merge 8 commits intoRust-Developers-JP:mainfrom
Conversation
|
今回はPRありがとうございます。 |
KaiTomotake
left a comment
There was a problem hiding this comment.
特に問題はなさそうです。
PRありがとうございました!
|
すみません、一つ問題が発生しまして・・・ |
|
@okakatsuo |
|
@KaiTomotake |
|
署名付きコミットへの書き換えが完了いたしました。 |
Signed-off-by: hinshi <105423175+hinshiba@users.noreply.github.com>
Signed-off-by: hinshi <105423175+hinshiba@users.noreply.github.com>
|
ご指摘ありがとうございます。 |
|
ありがとうございます. |
hinshiba
left a comment
There was a problem hiding this comment.
構造体への実装のexに日本語の追加の説明があって丁寧で非常に良いと思います.
ただ,1つめの例は誤解を招きそうに思えました.
意見を聞かせていただけると幸いです.
| let v = vec![0, 1, 2]; | ||
| // vの所有権がiterに移動する | ||
| let iter: std::vec::IntoIter<i32> = v.into_iter(); | ||
|
|
||
| // 所有権を持っているので、要素を直接消費できる | ||
| for x in iter { | ||
| println!("{}", x); | ||
| } |
There was a problem hiding this comment.
この例はやや誤解を招くように思えます.
ここで要素を消費しているのはforだと思いますが,println!()があるのでそちらが消費するように見えてしまうと思うのですが,どうでしょうか?
もちろんforにコメントがついていると言われればそれまでですが
KaiTomotake
left a comment
There was a problem hiding this comment.
申し訳ありません。ついこの間追加したルールで「サポートされているバージョンを明記する」というものがあります。もう一度vecモジュールの部分を見てもらいそれを参考にして、バージョンセクションを追記してもらえればと思います。
概要
構造体
std::vec::IntoIterのリファレンスページの新規作成とvecモジュール内の既存のタイポ修正を行いました。変更内容
src/vec/IntoIter.mdを作成。SpliteをSpliceに修正。InteratorをIteratorに修正。関連Issue
チェックリスト
mdbook buildを実行して確認したレビューで見てほしい点
IntoIterの記述ルールが、テンプレートであるvecモジュールの形式と一致しているか。