This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: [RFA] windows-nat.c: Enable processed input at startup
- From: Christopher Faylor <cgf-use-the-mailinglist-please at sourceware dot org>
- To: gdb-patches at sourceware dot org, Pierre Muller <pierre dot muller at ics-cnrs dot unistra dot fr>
- Date: Mon, 26 Apr 2010 10:32:41 -0400
- Subject: Re: [RFA] windows-nat.c: Enable processed input at startup
- References: <005401cae535$61adadc0$25090940$@muller@ics-cnrs.unistra.fr>
On Mon, Apr 26, 2010 at 01:41:16PM +0200, Pierre Muller wrote:
> This is the patch I was talking about in
>a reply to this thread:
>http://sourceware.org/ml/gdb/2010-04/msg00113.html
>
> It ensures that at startup, if GDB can connect to the console
>the console mode is modified to enable processed input.
> This make '^C' (ascii #3) to be interpreted as a
>control event.
>
>
>
>
>Pierre Muller
>Pascal language support maintainer for GDB
>
>
>2010-04-26 Pierre Muller <muller@ics.u-strasbg.fr>
>
> * windows-nat.c (_initialize_windows_nat): Try to set
> ENABLE_PROCESSED_INPUT for console mode if accessible.
>
>Index: windows-nat.c
>===================================================================
>RCS file: /cvs/src/src/gdb/windows-nat.c,v
>retrieving revision 1.208
>diff -u -p -r1.208 windows-nat.c
>--- windows-nat.c 16 Apr 2010 07:49:35 -0000 1.208
>+++ windows-nat.c 26 Apr 2010 11:33:08 -0000
>@@ -2356,6 +2356,22 @@ void
> _initialize_windows_nat (void)
> {
> struct cmd_list_element *c;
>+ HANDLE std_in_handle;
>+
>+ /* Try to enable processed input for the console.
>+ This should allow to use '^C' to interrupt the debuggee
>+ at least as log as the debugge does not modify the
>+ console mode settings. */
>+ std_in_handle = CreateFile ("CONIN$", GENERIC_READ | GENERIC_WRITE,
>+ FILE_SHARE_READ, NULL, OPEN_EXISTING, 0, 0);
>+
>+ if (std_in_handle != INVALID_HANDLE_VALUE)
>+ {
>+ DWORD console_mode;
>+ CHECK (GetConsoleMode (std_in_handle, &console_mode));
>+ console_mode = console_mode | ENABLE_PROCESSED_INPUT;
>+ CHECK (SetConsoleMode (std_in_handle, console_mode));
>+ }
>
> init_windows_ops ();
That has to be conditional on __CYGWIN__ since you'll be screwing up Cygwin's
notion of the console state.
And, the name std_in_handle is a misnomer. It should be console_handle
or something like that.
Can't you just set what you need and close the handle? I thought this
setting was global.
cgf