enum 先行宣言 (ECodeType)

[gorogoro123]

PR対象

• アプリ(サクラエディタ本体)

カテゴリ

• 改善

PR の背景

Clang/LLVM で build時 に "-Wtautological-constant-out-of-range-compare" のwarningが出力される。
CFileLoad.cpp(299,44): warning : result of comparison of constant 65000 with expression of type 'ECodeType' is always true [-Wtautological-constant-out-of-range-compare]

仕様・動作説明

1. enum ECodeType に基底型を追加します。
2. StdAfx.h から #include "env/DLLSHAREDATA.h" コメントアウト時の build error を修正します。

PR の影響範囲

影響なし。

テスト内容

1. Clang/LLVM build 時に "-Wtautological-constant-out-of-range-compare" の warning が削減されていることを確認する。
2. 変更前後でasmが同じことを確認する。

関連 issue, PR

#1615

参考資料

https://en.cppreference.com/w/cpp/language/enum

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[AppVeyorBot]

✅ Build sakura 1.0.4467 completed (commit 212015e298 by @gorogoro123)

[berryzplus]

主題「前方宣言しないと怒られる」の対策は以下で解決できそうに思います。

Suggested change

	enum ECodeType : int;
	#include "charset/charset.h"

主題以外の修正が多いですが、「ダメ」でない気はしてます。