-
Notifications
You must be signed in to change notification settings - Fork 168
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
C/C++の強調キーワード更新 #1102
C/C++の強調キーワード更新 #1102
Conversation
✅ Build sakura 1.0.2404 completed (commit 64b95b565a by @) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
最近の規格も含めて、C/C++でキーワードになりうる単語が初期の強調リストに入る変更と理解しました。
とくに問題ないと思います。
L"const", | ||
L"const_cast", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
消しているわけではなくて、辞書順になるように記述箇所を移動しただけですね。
一瞬 const_cast
は C++ のキーワードじゃない、という主張かと思いましたけど 😃
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
差分を少なくするために余計な変更はしない方が良かったかもしれませんがやっちゃいました。
Cプリプロセッサのキーワードは他のC/C++のキーワードとは記述順序を分けるとかも考えましたがそれはやりませんでした。
/azp run |
Azure Pipelines successfully started running 1 pipeline(s). |
レビューありがとうございました。Merge します。 |
C/C++の強調キーワード更新
PR の目的
C/C++の強調キーワードセットの初期値を更新しました。
カテゴリ
PR の背景
サクラエディタは様々なタイプ別設定を扱う事が出来て、タイプ別設定毎に強調キーワードを指定する事が出来ます。
PR のメリット
強調キーワードセットに最初から色々なキーワードが含まれていれば、各ユーザーが追加する必要が無い。
PR のデメリット (トレードオフとかあれば)
時と場合によっては強調してほしくないキーワードが追加されているかもしれない。
関連チケット
#64, #801, #1092
参考資料
https://en.cppreference.com/w/cpp/keyword