Skip to content

Conversation

@gorogoro123
Copy link
Owner

PR対象

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

カテゴリ

  • 改善

PR の背景

Unicode版のみ対応だが、tchar.h をincludeしている。
これまでの対応が不十分で、tchar.h を使用する関数が残っていた。

仕様・動作説明

  1. _T() マクロ(tchar.h) を TEXT() マクロ (winnt.h) に置き換える。
  2. include <tchar.h> を削除する。

PR の影響範囲

影響なし。

テスト内容

  1. コンパイル設定で、プリプロセッサ - ファイル前の処理 - はい(/P) に設定、前処理済み出力をファイル(*.i)に書き込み、変更前後で同じであることを確認する。
    Debug/Win32 build 時のプリプロセッサ処理結果
CShareData::InitShareData()
std::wstring strShareDataName = GSTR_SHAREDATA;
std::wstring strShareDataName = (L"SakuraShareData" L"" L"WP" L"_DEBUG" L"181");

CNormalProcess::_GetInitializeMutex()
std::wstring strMutexInitName = GSTR_MUTEX_SAKURA_INIT;
std::wstring strMutexInitName = (L"MutexSakuraEditorInit" L"" L"WP" L"_DEBUG" L"181");
  1. 変更前後でasmが同じことを確認する。

関連 issue, PR

参考資料

https://learn.microsoft.com/ja-jp/windows/win32/api/winnt/nf-winnt-text

@gorogoro123 gorogoro123 force-pushed the feature/remove_include_tchar_h1 branch from 678fc08 to 3784198 Compare July 9, 2025 12:35
@gorogoro123 gorogoro123 force-pushed the feature/remove_include_tchar_h1 branch from 3784198 to e22f1b2 Compare July 12, 2025 03:07
@gorogoro123 gorogoro123 force-pushed the master branch 4 times, most recently from eef5572 to 4b096cd Compare November 16, 2025 04:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants