Closed HashidaTKS closed 3 months ago
以下の警告を解消。
C:\gitdir\Chronos\CTabWnd.h(8): warning C26812: 列挙型 'ETabWndNotifyType' は対象範囲外です。'enum' (Enum.3) より 'enum class' を優先します。
これらはコマンドメッセージのIDとして使われている。 少なくともChronosではメッセージIDはほぼマクロで定義されているため、マクロで定義するように変更。 (enum classにすると、コマンドのメッセージとして渡すときは必ず数値型にしないといけないし、少し微妙な感じがするということもある)
https://github.com/ThinBridge/Chronos/pull/173 と違って単なる変数にしていないのは、上記のようにメッセージはマクロで定義されていることが多いことと、スコープが狭い名前空間で定義されているわけでもないため。
警告の解消の確認
リグレッションテスト
タブの動作に関するソースコードに変更が入っているので、タブの動作を確認する。
以下の警告が表示されないこと
Which issue(s) this PR fixes:
42
What this PR does / why we need it:
以下の警告を解消。
これらはコマンドメッセージのIDとして使われている。 少なくともChronosではメッセージIDはほぼマクロで定義されているため、マクロで定義するように変更。 (enum classにすると、コマンドのメッセージとして渡すときは必ず数値型にしないといけないし、少し微妙な感じがするということもある)
https://github.com/ThinBridge/Chronos/pull/173 と違って単なる変数にしていないのは、上記のようにメッセージはマクロで定義されていることが多いことと、スコープが狭い名前空間で定義されているわけでもないため。
How to verify the fixed issue:
The steps to verify:
警告の解消の確認
リグレッションテスト
タブの動作に関するソースコードに変更が入っているので、タブの動作を確認する。
Expected result:
警告の解消の確認
以下の警告が表示されないこと
リグレッションテスト