doc-enの差分を反映(1) #138
Merged
doc-enの差分を反映(1) #138
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
zonuexe
commented
Oct 31, 2024
| が空の文字列(<literal>""</literal>)以外に設定されているとき、 | ||
| <link xlink:href="&url.rfc;4180">RFC 4180</link> | ||
| に準拠しない CSV が生成されたり、PHP の CSV | ||
| 関数を介してラウンドトリップ(往復変換)でデータが壊れる可能性があります。 |
Contributor
Author
There was a problem hiding this comment.
ここが翻訳に迷うところで、直訳すると「ラウンドトリップを生き残ることができない」。
Contributor
There was a problem hiding this comment.
つまり、例えば「CSVをPHPのcsv関数(fgetcsvとかstr_getcsvとか)で読み込んで、fputcsvとかで書き込んだときの内容が保証されない」という意味ですよね。round tripは往復の旅行ですから、それをCSVのデータに例えているということですかね。
zonuexe
commented
Oct 31, 2024
| 簡単に習得することができます。この言語は、動的に生成されるウェブページを | ||
| Web 開発者が速やかに作成できるようにすることを主な目標として | ||
| つくられました。しかし、それだけにとどまらず、 | ||
| もっと多くのことを PHP を使って行うことができます。 |
Merged
zonuexe
force-pushed
the
follow/doc-en-1
branch
from
eabdf7b to
f2e751e
Compare
Contributor
Author
|
ビルドが失敗しているように見えるのは php/doc-en@4cb53ec が影響してそうに見えるので別ブランチで作業を進めます |
youkidearitai
approved these changes
Oct 31, 2024
Contributor
youkidearitai
left a comment
youkidearitai
left a comment
There was a problem hiding this comment.
LGTM!
ラウンドトリップの件で特になければ、おいおいマージさせていただきます。
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
以下のコミットの変更に追従しました