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

[WIP] CNativeWにnullptrを代入できるようにしたい Take3 #1097

Conversation

berryzplus
Copy link
Contributor

PR の目的

CNativeWの挙動を仕様化した際、疑問があった箇所に「バグじゃね?」とコメントしました。
現時点で、「バグじゃね?」が2箇所残っているので対応したいです。

疑問のある仕様

  • CNativeWにNULLを代入したとき、CNativeWのデータは空文字列になる。
  • CNativeWにNULLを加算したとき、CNativeWのデータ末尾にNUL文字が追加される。

カテゴリ

  • その他

PR の背景

自分のなかで結論が出せてないので、作成途中のものを一旦公開します。

PR のメリット

  • str = NULL という記述の意味が、str をNULLにする、になります。
    • 従来は str を空文字列にしていました。
  • str += NULL という記述の意味が、 なにもしない になります。
    • 従来は str の末尾にNUL文字を加算していました。

PR のデメリット (トレードオフとかあれば)

  • operator = (wchar_t) の存在意義を問うべきなのかもしれません。
  • str += NULL という記述の意味が、従来通りで正しい可能性があります。

PR の影響範囲

  • アプリ(=サクラエディタ)の機能に影響はありません。

関連チケット

@AppVeyorBot
Copy link

@berryzplus
Copy link
Contributor Author

#1098 たてたので閉じます。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-changelog 【ChangeLog除外】
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants