Skip to content
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

Fix Typo in String #2119

Closed
wants to merge 1 commit into from
Closed

Fix Typo in String #2119

wants to merge 1 commit into from

Conversation

@Kanezoh
Copy link
Contributor

Kanezoh commented Jan 29, 2020

他の部分では"null文字"と書かれていますが、最初だけ”NUL”になっていました。
表記を統一した方が良いと思われます。

Copy link
Member

hanachin left a comment

表記を合わせたほうがよいのには賛成です 👍

PRのタイトルについてコメントしますが、typoではないと思います

ヌル文字(または「ヌル終端文字」)とは、値0の制御文字である。NULと表記される場合もある[1]。
https://ja.wikipedia.org/wiki/%E3%83%8C%E3%83%AB%E6%96%87%E5%AD%97

"null文字"はこの refm/api/src/_builtin/pack-template でしか使われてないのかな?

ASCII文字列(null文字を詰める/後続するnull文字やスペースを残す)

どっちの方に合わせたほうがいいだろう

@znz

This comment has been minimized.

Copy link
Member

znz commented Jan 29, 2020

null は null pointer などの意味もあって読む時にちょっと悩みますが、 NUL は他の意味で使われていなくて明確なので、個人的には ASCII の制御文字由来の NUL 文字の方に統一したいです。

Copy link
Member

hanachin left a comment

Google検索した結果、次の通り「ヌル文字」が一番多く使われていそうなので「ヌル文字」にあわせるのはどうでしょうか

"ヌル文字" - "ヌル文字列" 40,200件
"NULL文字" -"NULL文字列" 約 37,200 件
”NUL文字" 約 7,160 件

refm/api/src/_builtin/pack-template では「ヌル文字」も既に使われているようですし

: テンプレート文字"a"は、長さが足りない分をヌル文字で補う

@Kanezoh

This comment has been minimized.

Copy link
Contributor Author

Kanezoh commented Jan 29, 2020

ご指摘ありがとうございます。
znzさんのご指摘にも納得できますが、やはり一般的な表記ではないようなので、私としてはヌル文字表記を支持したいです。
PRのタイトルの誤りの件も含めて、新しくプルリク #2120 を作成しましたのでよろしくお願いします。

@Kanezoh Kanezoh closed this Jan 29, 2020
@Kanezoh Kanezoh deleted the Kanezoh:fix_typo_in_string branch Jan 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.