Skip to content

fix(fdc): remove ReadDiagnostic OSD toast during gameplay - #30

Merged
bubio merged 1 commit into
mainfrom
remove-read-diagnostic-toast
Aug 30, 2026
Merged

bubio merged 1 commit into
mainfrom
remove-read-diagnostic-toast

Conversation

@bubio

@bubio bubio commented Aug 30, 2026

Copy link
Copy Markdown
Owner

Summary

  • FDC の Read Diagnostic コマンド実行時に表示していた ReadDiagnostic (DrX TrY) のOSDトーストを削除
  • 他のFDCコマンド (Read/Write/ReadID/WriteID) は「Show FDC Status」設定が有効な時のみ表示されるが、ReadDiagnostic だけは無条件に表示されており、ソーサリアンのオープニングなどで通常プレイ中にゲーム画面へ被って表示されてしまう問題があった
  • ディスクマウント失敗など、ユーザーに伝えるべき他のOSDメッセージ表示の仕組み自体には手を加えていない

Test plan

  • cmake --build . でビルドが通ることを確認
  • src/pc88/fdc.cpp の変更差分が意図通り(ReadDiagnostic用の statusdisplay.Show() 呼び出しとトラック番号算出のみ削除)であることを確認

🤖 Generated with Claude Code

https://claude.ai/code/session_01Rg91L7dcSjpYirGT9hWZXU

Read Diagnostic (FDC command 2) is a routine track-level disk access
some titles use for verification/copy-protection checks. It was being
surfaced via statusdisplay.Show() unconditionally, unlike other FDC
commands that gate behind the "Show FDC Status" option, so it kept
popping up as an on-screen toast during normal gameplay (e.g. the
Sorcerian opening) with no way to suppress it.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Rg91L7dcSjpYirGT9hWZXU
@bubio
bubio merged commit 382daa4 into main Aug 30, 2026
16 checks passed
@bubio
bubio deleted the remove-read-diagnostic-toast branch August 30, 2026 09:57
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.

1 participant