SonarQubeの静的解析で検出されたFileNameSepExtのバグを修正する（その２）

[berryzplus]

PR の目的

SonarCloudにバグだと言われているコードを修正し、見た目の品質を上げます。

カテゴリ

• リファクタリング

※検知されたのは「nullチェック漏れによるクラッシュの可能性」ですが、
　実態として実害はないと考えられるためリファクタリングとします。

PR の背景

SonarCloudにより以下の不具合が検出されています。
https://sonarcloud.io/project/issues?id=sakura-editor_sakura&issues=AXhxfE2dTNlLrH0R7vj9&open=AXhxfE2dTNlLrH0R7vj9

utif/file.cpp 内のローカル関数 FileNameSepExt の第一引数 file に nullptr が渡された場合の考慮がないよ（≒渡ってきたら落ちるよ）という指摘です。

PR のメリット

• SonarCloudにバグと検出される残念なコードが1つ消えます。

PR のデメリット (トレードオフとかあれば)

仕様・動作説明

アプリの仕様・機能に変更はありません。

• FileNameSepExt 　＝　問題が検出された関数です。
  • シグニチャを変更して file に nullptr を渡せないようにします。
  • シグニチャを変更してout引数のバッファサイズを考慮しなくて良いようにします。
• FileMatchScoreSepExt 　＝　FileNameSepExtの呼出元関数
  • シグニチャを変更して file1 と file2 に nullptr を渡せないようにします。
  • バッファ変数の定義型をWCHAR[260] から std::wstring に変更します。

PR の影響範囲

Diffダイアログに表示するファイル名比較に使われる関数です。

テスト内容

このPRは、修正部分のテストを含んでいます。

テスト1

手順

関連 issue, PR

• SonarCloudにBugだと言われているものを対処したいなぁという話。 #1504
• SonarQubeの静的解析で検出されたFileNameSepExtのバグを修正する #1796

参考資料

[AppVeyorBot]

✅ Build sakura 1.0.4030 completed (commit 77ac75dbb5 by @berryzplus)

[AppVeyorBot]

✅ Build sakura 1.0.4031 completed (commit 161a888b8a by @berryzplus)

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

100.0% Coverage
0.0% Duplication

[kengoide]

問題なさそうに見えます。

[berryzplus]

レビューありがとうございます。マージしてしまいます。
何か不都合あれば別PRで対応していきたいと思います。