Skip to content

4.17. String #69

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

4.17. String #69

merged 5 commits into from
Feb 7, 2016

Conversation

pocket7878
Copy link
Contributor

No description provided.

@pocket7878 pocket7878 changed the title [WIP] String String Feb 6, 2016
@pocket7878
Copy link
Contributor Author

添削宜しくお願い致します @KeenS

文字列は、プログラマーがマスターすべき重要なコンセプトです。
Rustの文字列の扱いは、Rust言語がシステムにフォーカスしているため、少し他の言語と異なります。
動的なサイズを持つデータ構造が有る時、常に物事は複雑になります、そして文字列もまたサイズを変更することができるデータ構造です。
これはつまり、Rustの文字列もまた、Cのような他のシステム言事は少し異なるということです。
Copy link
Member

Choose a reason for hiding this comment

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

x 言事は
o 言語とは

@pocket7878
Copy link
Contributor Author

@KeenS 添削ありがとうございます。 null-terminatedは、{null or NULL}終端 といった感じで訳されてると思います、nullをどの形で訳するかはいろいろ有ると思います。とりあえず、すべて小文字のnullにしてみました。

@KeenS
Copy link
Member

KeenS commented Feb 6, 2016

確認しました。後程マージします。お疲れ様でした。

@tatsuya6502 tatsuya6502 changed the title String 4.17. String Feb 7, 2016
KeenS added a commit that referenced this pull request Feb 7, 2016
@KeenS KeenS merged commit 403d47b into rust-lang-ja:master Feb 7, 2016
@Nnwww
Copy link
Contributor

Nnwww commented Feb 8, 2016

@pocket7878 @KeenS 成果物巡回中に細かいながらも幾つか抜けがあったためコメントさせて頂きました。

@KeenS
Copy link
Member

KeenS commented Feb 8, 2016

@Nnwww
ありがとうごさいます。

closedなpull reqにコメントを付けると見逃しかねないので新規にイシューかプリルクを上げていただけると幸いです

@Nnwww
Copy link
Contributor

Nnwww commented Feb 8, 2016

すみません以後気をつけます。PR: #74

@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