TCHAR置き換え

[gorogoro123]

PR対象

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

カテゴリ

• 改善

PR の背景

TCHARからWCHARに置き換えられましたが、まだTCHARが残っています。

仕様・動作説明

残りのTCHARをWCHARに置き換えます。

PR の影響範囲

影響なし。

テスト内容

1. 変更前後のasmを比較して同じであることを確認する。
2. "TCHAR"を検索して、残ってないことを確認する。
3. 型チェックする。
   static_assert(std::is_same<WCHAR, WIN_CHAR>::value);
   static_assert(std::is_same<WCHAR, TCHAR>::value);

関連 issue, PR

#1034

参考資料

https://learn.microsoft.com/ja-jp/windows/win32/learnwin32/working-with-strings

[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.4386 completed (commit a518d1c7ed by @gorogoro123)

[beru]

ざっとしか見てませんが問題無いと思います。

ところで TCHAR を使わずに WCHAR を使うのであればマクロ経由にせずに末尾WのWindowsAPIを直接呼び出しても良いのかも？と思いました。もちろんこのPRで行う必要はなく、今後本当にやるべきかもよくわかりません。