使用されていないCRecentの呼び出し箇所を除去する

[ghost]

PR の目的

CRecentの不要な呼び出し箇所を除去し、当該クラス・関数がCRecentに依存しないことを明確にする

カテゴリ

• リファクタリング

PR の背景

#1697 の対応時にCRecentの利用状況を確認したところ、未使用にも関わらずインスタンスを保持している箇所がありました。
このうち、CDlgOpenFileDataについては、 #1481 のマージ以降未使用になっています。

これら未使用変数の除去を提案するものです。

PR のメリット

今回変更対象となるクラス・関数とCRecentの間にあった依存関係を取り除くことができます。

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

特にありません。

仕様・動作説明

• CDlgOpenFileDataのメンバから、CRecentFile/CRecentFolderを取り除きました。
• CTabWnd::TabWindowNotify()から、未使用のローカル変数となっているCRecentEditNodeを取り除きました。

PR の影響範囲

• ファイル選択ダイアログ
• タブウィンドウ

テスト内容

動作に変更がなく、使用されていない変数宣言を削除したのみであるため、ビルドの成否を確認できれば良いと思います。

関連 issue, PR

#1481
#1697

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information

[AppVeyorBot]

✅ Build sakura 1.0.3852 completed (commit b16bfb8b2a by @kazasaku)

[kengoide]

対応してくださりありがとうございます。問題ないものと思います。

[ghost]

レビューありがとうございました。
他にコメントなどがなければこのままマージしてしまいます。

[ghost]

マージしました。