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

change null文字 to ヌル文字 #2120

Merged
merged 2 commits into from Jan 30, 2020
Merged

Conversation

@Kanezoh
Copy link
Contributor

Kanezoh commented Jan 29, 2020

#2119 の修正版です。
文字列におけるNUL文字,null文字表記をヌル文字に統一しました。
null終端文字列も同様にヌル終端文字列に変更しました。

@Kanezoh Kanezoh mentioned this pull request Jan 29, 2020
Copy link
Member

hanachin left a comment

Pull Requestありがとうございます 👍

refm/api/src/_builtin/String には「ナルキャラクタ」の表記もあるようですね。

シンボル文字列にはナルキャラクタ("\0")、空の文字列の使用も可能です。

他にも refm/api/src/_builtin/pack-template では「ナル終端」とか「ナルバイト」のような言葉が使われていそう

@Kanezoh

This comment has been minimized.

Copy link
Contributor Author

Kanezoh commented Jan 30, 2020

ナルも使われていたのですね...
見落とし失礼しました。
これは単に読み方の問題なのでそのままヌルで置き換えて良さそうですね。

@Kanezoh

This comment has been minimized.

Copy link
Contributor Author

Kanezoh commented Jan 30, 2020

ナルヌルに変更しました。
null表記はnullポインタを表す際しか使われていないのでこれで表記が明確になったと思います。

Copy link
Member

hanachin left a comment

念の為"ナル文字"でもGoogle検索してみました、7,810 件でした。
一番使われていそうな表記に統一されたので、よさそう 🌾

@hanachin hanachin merged commit 9a1d17b into rurema:master Jan 30, 2020
8 checks passed
8 checks passed
rake
Details
misspell
Details
Pages changed 28 new files uploaded
Details
Redirect rules No redirect rules processed
Details
Header rules 1 header rule processed
Details
Mixed content No mixed content detected
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
deploy/netlify Deploy preview ready!
Details
@Kanezoh Kanezoh deleted the Kanezoh:change_null_notation branch Jan 31, 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

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