MinGW ビルドでややおかしなメッセージが出る。

[ds14050]

無題テキストを変更して終了しようとすると「(1 は変更されています」と表示されます。Win32 ビルドでは master も 2.3.2.0 も「(無題)1 は変更されています」と表示されます。

MinGW ビルドをリリースしない限りは問題ではないともいえます。

[berryzplus]

う～む・・・。

調べ方）

無題テキストを変更して終了しようとすると「(1 は変更されています」と表示されます。Win32 ビルドでは master も 2.3.2.0 も「(無題)1 は変更されています」と表示されます。

「メッセージがおかしい」という問題に対する調査は、メッセージ現物を調べるのが基本です。
そこで、申告のあったメッセージの共通部分「は変更されています」でソース全体をgrepします。

vs2017(英語版)での検索結果がこちらです。

Find all "は変更されています", Regular expressions, Subfolders, Keep modified files open, Find Results 2, Entire Solution, "*.*"
  C:\work\sakura-editor\sakura_core\sakura_rc.rc(3209):	STR_ERR_DLGEDITDOC30		"%ts\nは変更されています。 閉じる前に保存しますか？\n\nビューモードで開いているので、名前を付けて保存すればいいと思います。\n"
  C:\work\sakura-editor\sakura_core\sakura_rc.rc(3211):	STR_ERR_DLGEDITDOC31		"%ts\nは変更されています。 閉じる前に保存しますか？"
  C:\work\sakura-editor\sakura_core\sakura_rc.rc(3314):	STR_ERR_CEDITVIEW_CMD18		"%ts\nは変更されています。 Oracle SQL*Plusで実行する前に保存しますか？"
  C:\work\sakura-editor\sakura_core\sakura_rc.rc(3324):	STR_ERR_CEDITVIEW_CMD29		"%ts\n\nこのファイルは変更されています。\n再ロードを行うと変更が失われますが、よろしいですか?"
  Matching lines: 4    Matching files: 1    Total files searched: 693

問題になっているのはおそらく STR_ERR_DLGEDITDOC31 です。
他のメッセージIDならばその旨説明があるはず、という仮定に基づく予想ですが。
ここまで分かればあとはメッセージがどこで使われているか見るだけです。

問題箇所はここ・・・のはず。

sakura/sakura_core/doc/CEditDoc.cpp

Lines 932 to 939 in f2af1b7

	else {
	nRet = ::MYMESSAGEBOX(
	hwndMainFrame,
	MB_YESNOCANCEL | MB_ICONQUESTION | MB_TOPMOST,
	GSTR_APPNAME,
	LS(STR_ERR_DLGEDITDOC31),
	pszTitle
	);

対策は独自のフォーマット書式型指定子「t(%ts, %tc)」の廃止かなぁ・・・
いまちょっと忙しいので再現とか検証とかできてない状況です。

何気にこっちの問題かも知れません。

sakura/sakura_core/doc/CEditDoc.cpp

Line 896 in f2af1b7

auto_sprintf( szGrepTitle, _T("%s%d"), LS(STR_NO_TITLE1), node->m_nId ); //(無題)

LS が返すのはクラス型 CLoadString なので 明示的に (LPCTSTR) してあげないと文字列にならないのかも。

[berryzplus]

なんか後者くさい気がしてきました。
細かい検証は任せます。ってか寝ます 😄

[berryzplus]

あかん、もっと深刻な話やん。

同種の障害が起こり得る場所をピックアップしてみました。

Find all "sprintf\(\s*\w+,\s*[^,\n]+,\s*\bLSW?\s*\(", Match case, Regular expressions, Subfolders, Keep modified files open, Find Results 2, Entire Solution, "*.*"
  C:\work\sakura-editor\sakura_core\doc\CEditDoc.cpp(896):		auto_sprintf( szGrepTitle, _T("%s%d"), LS(STR_NO_TITLE1), node->m_nId );	//(無題)
  C:\work\sakura-editor\sakura_core\env\CFileNameManager.cpp(471):			wsprintf( szFileName, _T("%s%d"), LS( STR_NO_TITLE1 ), nId);
  C:\work\sakura-editor\sakura_core\prop\CPropComBackup.cpp(524):			auto_sprintf( temp, LTEXT("%ls\\"), LSW(STR_PROPCOMBK_DUSTBOX) );
  Matching lines: 3    Matching files: 3    Total files searched: 693

[berryzplus]

興味ある人も居ないようなので閉じておきます。

必要あれば新たにissue立ててください。