ThinBridge / Chronos

Chromiumベースの業務ブラウザ
Other
5 stars 3 forks source link

Suppress warning C26812 of ResizeSpecifier #173

Closed HashidaTKS closed 3 months ago

HashidaTKS commented 3 months ago

Which issue(s) this PR fixes:

https://github.com/ThinBridge/Chronos/issues/42

What this PR does / why we need it:

以下の警告を解消。

C:\gitdir\Chronos\DlgSetting.h(17): warning C26812: 列挙型 'autoresize::ResizeSpecifier' は対象範囲外です。'enum' (Enum.3) より 'enum class' を優先します。

値の比較やビット演算をしている箇所などの変更を極力抑えるよう、enum classではなく名前空間autoresize上の単なる変数とすることで対処。

How to verify the fixed issue:

The steps to verify:

警告の解消の確認

  1. Code Analysisを実行する。また、Chronosをリビルドする。

リグレッションテスト

※変更したクラスは上記のようなダイアログでリサイズ時に呼ばれる

Expected result:

警告の解消の確認

以下のWarningが出ないこと。

C:\gitdir\Chronos\DlgSetting.h(17): warning C26812: 列挙型 'autoresize::ResizeSpecifier' は対象範囲外です。'enum' (Enum.3) より 'enum class' を優先します。

また、Chronosのビルドに成功すること。

リグレッションテスト