CMacro.cppでのconst_castをやめる

[suconbu]

PR対象

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

カテゴリ

• リファクタリング

PR の背景

#1948 の修正を行った時にSonarCloudで指摘が出た箇所 (下記リンク) に関する修正になります。
https://sonarcloud.io/project/issues?resolved=false&sinceLeakPeriod=true&pullRequest=1948&id=sakura-editor_sakura

修正方法としては、

• CMacro::HandleFunction とそれに関連する関数の引数 (Arguments) から const を取り除いた上で、
• CMacro.cpp で VariantChangeType を呼び出している箇所の const_cast をすべて削除します。

既存コードの VariantChangeType の呼び出し箇所に const_cast が付いているのは、同じ VariantChangeType でもMinGW環境では第2引数に const が付かないプロトタイプとなっているためのようでした。

Msvc (oleauto.h)

WINOLEAUTAPI VariantChangeType(_Inout_ VARIANTARG * pvargDest,
               _In_ const VARIANTARG * pvarSrc, _In_ USHORT wFlags, _In_ VARTYPE vt);

MinGW (oleauto.h)

WINOLEAUTAPI VariantChangeType(VARIANTARG *pvargDest,VARIANTARG *pvarSrc,USHORT wFlags,VARTYPE vt);

仕様・動作説明

静的解析の指摘修正のみのため省略します。

PR の影響範囲

特にないと思います。

テスト内容

SonarCloudの指摘がなくなることを確認します。

関連 issue, PR

#1948

参考資料

[sonarcloud]

Quality Gate failed

Failed conditions
2.8% Coverage on New Code (required ≥ 80%)

See analysis details on SonarCloud

[AppVeyorBot]

✅ Build sakura 1.0.4339 completed (commit b11ab9df93 by @suconbu)

[suconbu]

レビューありがとうございます。
先に本PRからマージしてしまおうと思います。