Re: [hatari-devel] Hatari debug mode |
[ Thread Index |
Date Index
| More lists.tuxfamily.org/hatari-devel Archives
]
- To: hatari-devel@xxxxxxxxxxxxxxxxxxx
- Subject: Re: [hatari-devel] Hatari debug mode
- From: Eero Tamminen <oak@xxxxxxxxxxxxxx>
- Date: Thu, 31 Jul 2025 12:23:11 +0300
- Cc: Christian Zietz <czietz@xxxxxxx>
- Dkim-filter: OpenDKIM Filter v2.11.0 smtp.dnamail.fi D7EBD4098E8B
- Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=helsinkinet.fi; s=2025-03; t=1753953791; bh=LO/LyfkCoOKJz3CYVkyrYYexPQQLN8d/zf4cUe9TqTk=; h=Date:Subject:To:References:Cc:From:In-Reply-To:From; b=eH8gzapx6RUf0qDurcbnpqubLBA5+urvJpcolPLciVnb6/vSFRE3QvX9QVJmlydzx dku8auYKlAm9E2l6CZkVORgIxhfraQ+abnY4RW84cBraHzBwW2SGPJJKXUeEncdHI8 G+xrNfCGbODMmy77xAnk7p4XTmIJBeXn6Ybqoya8wmcC1a1EzqBkQXL74LPVfOaZvD jD2JcbO5HA9eFvFBmks1cQ2mZ6l6Cz7BQYgkpT14tWFO1yVCYsb6NQKfq1CYb3INdV lytNa78uiw6T8P3CS23nceLuKXHuNScDAQ3Z9vevDA60HYTWmA0t209wpd+yzxd/Mx oSuIB66UD66qQ==
Hi,
On 30.7.2025 20.41, Christian Zietz wrote:
PS: I could potentially test it earlier if I get a (pipeline-provided)
binary. Then again, I can imagine that you don't want to push untested
changes to the repo so close to a new release...
Although, even without testing, I wonder if the patch is really doing
what you (Eero) probably intended.
DebugUI_Init() is called unconditionally at the end of Main_Init():
https://framagit.org/hatari/hatari/-/
blob/7772819bc1191065b3ee9c055967b239eeab9d51/src/main.c#L855
... not just when the user presses AltGr+Pause. Wouldn't the patch
therefore *always* open a console on Windows?
Thanks, good catch!
Attached is a better patch.
- Eero
From 0d28c24e3665e9316acc821b08df9d943f506d0f Mon Sep 17 00:00:00 2001
From: Eero Tamminen <oak@xxxxxxxxxxxxxx>
Date: Mon, 28 Jul 2025 02:30:58 +0300
Subject: [PATCH] Open Windows console automatically for Debugger
---
src/debug/debugui.c | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/src/debug/debugui.c b/src/debug/debugui.c
index 563a87d4..7519188d 100644
--- a/src/debug/debugui.c
+++ b/src/debug/debugui.c
@@ -1231,6 +1231,10 @@ void DebugUI(debug_reason_t reason)
if (welcome)
{
+#ifdef WIN32
+ /* in case user forgot -W option */
+ Win_ForceCon();
+#endif
fputs(welcome, stderr);
welcome = NULL;
}
--
2.39.5