Merge pull request #377 from sempreff/bugfix/#376-avoidLogfileOverwri…

…tePrompt avoid meaningless overwrite prompt when choosing log file
TeraTermProject · Oct 20, 2024 · f94aa83 · f94aa83
2 parents c0b1423 + 75d47b4
commit f94aa83
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 0 deletions.
diff --git a/doc/en/html/about/history.html b/doc/en/html/about/history.html
@@ -60,6 +60,7 @@ <h3 id="teraterm_5.4">YYYY.MM.DD (Ver 5.4 not released yet)</h3>
       <li>Fixed crash problem when Realtime mode is off and History is on, on <a href="../menu/control-broadcast.html">Broadcast command dialog</a>.
       <li>Fixed a problem that a left click on backgrounded terminal window makes text selection 1 char sensitively.</li>
       <li>MACRO: Improved the issue where DPI scaling was not properly adjusted when the dialog spans across monitors with different scaling factors.</li>
+      <li>Unnecessary overwrite confirmation dialog is no longer displayed when selecting a existing file as a log file at file selecting dialog box.</li>
     </ul>
   </li>
 

diff --git a/doc/ja/html/about/history.html b/doc/ja/html/about/history.html
@@ -60,6 +60,7 @@ <h3 id="teraterm_5.4">YYYY.MM.DD (Ver 5.4 not released yet)</h3>
       <li><a href="../menu/control-broadcast.html">Broadcast command ダイアログ</a>でリアルタイムモードが off 、ヒストリが on のとき、コマンドの履歴を保存する時にクラッシュする問題を修正した。</li>
       <li>バックグラウンドにある teraterm ウィンドウを左クリックで前面化した時に過敏に 1キャラクタ選択されてしまう問題を修正した。</li>
       <li>MACRO: ダイアログが拡大/縮小率の異なるモニタをまたぐ時、DPI調整がうまく行われない問題を改善した。</li>
+      <li>ログファイルとしてファイル選択ダイアログで既存のファイルを指定した時に上書き確認ダイアログを表示しないようにした。</li>
     </ul>
   </li>
 

diff --git a/teraterm/common/ttcommdlg.cpp b/teraterm/common/ttcommdlg.cpp
@@ -344,6 +344,10 @@ static BOOL GetOpenSaveFileNameW(const TTOPENFILENAMEW *ofn, bool save, wchar_t
 
 	DWORD options;
 	pFile->GetOptions(&options);
+	// 呼び出し側で未設定ならばクリア
+	if (!(OFN_OVERWRITEPROMPT & ofn->Flags)) {
+		options &= ~FOS_OVERWRITEPROMPT;
+	}
 	pFile->SetOptions(options | FOS_NOCHANGEDIR);
 
 	// 初期フォルダ