From: Mark Mitchell <mark@codesourcery.com>
To: Eli Zaretskii <eliz@gnu.org>
Cc: drow@false.org, bug-readline@gnu.org, gdb-patches@sources.redhat.com
Subject: Re: PATCH: Use getche on Win32
Date: Fri, 13 May 2005 11:42:00 -0000 [thread overview]
Message-ID: <42844946.4090202@codesourcery.com> (raw)
In-Reply-To: <01c555fd$Blat.v2.4$f4d65060@zahav.net.il>
Eli Zaretskii wrote:
>>Date: Tue, 10 May 2005 23:54:46 -0700
>>From: Mark Mitchell <mark@codesourcery.com>
>>CC: Eli Zaretskii <eliz@gnu.org>, bug-readline@gnu.org,
>> gdb-patches@sources.redhat.com
>>
>>As Daniel says, the Ctrl-C issue is independent of this code, and, as
>>Chris says, it's perfectly possible to capture Ctrl-C under Windows; we
>>just don't want to do it here.
>
>
> In the last sentence, what does ``here'' mean? Do you mean you don't
> want to do it in Readline? If so, I think it's a mistake: the Unix
> code of Readline clearly does manipulate signals, and I think a
> Windows port should be as close to it as it can.
You are of course correct that readline manipulates signals. In
particular, it registers signal handlers at the start of the function
named readline, and then removes them before returning. It would indeed
be desirable that the Windows version behave as much like POSIX as
possible in this respect.
However, when you said earlier that handling Ctrl-C was very important
to GDB, Daniel and I understood you to be referring to the ability to
interrupt the inferior when readline is not active. In other words,
after you say "run", you may want to hit Ctrl-C before you reach a
breakpoint. That functionality is independent of readline; it comes
from signal handlers that GDB itself installs.
So, what I wrote is way too strong a statement. I should have just said
that handling Ctrl-C here is not as critical as elsewhere. I also can't
find any suggestion that readline actually reads the "Ctrl" and "C"
characters through its main input loop; instead, I think that it just
installs signal handlers in the usual way. It's possible that I've just
not find the right bit yet, though.
Thanks,
--
Mark Mitchell
CodeSourcery, LLC
mark@codesourcery.com
(916) 791-8304
next prev parent reply other threads:[~2005-05-13 6:29 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-05-09 20:42 Mark Mitchell
2005-05-10 6:01 ` Eli Zaretskii
2005-05-10 11:45 ` Mark Mitchell
2005-05-10 20:05 ` Eli Zaretskii
2005-05-10 20:26 ` Mark Mitchell
2005-05-10 20:31 ` Eli Zaretskii
2005-05-10 20:49 ` Daniel Jacobowitz
2005-05-10 21:41 ` Christopher Faylor
2005-05-11 6:55 ` Christopher Faylor
2005-05-11 7:13 ` Eli Zaretskii
2005-05-11 7:17 ` Mark Mitchell
2005-05-11 7:21 ` MinGW build instructions (was: PATCH: Use getche on Win32) Eli Zaretskii
2005-05-11 7:29 ` MinGW build instructions Mark Mitchell
2005-05-11 7:53 ` Eli Zaretskii
2005-05-11 18:45 ` Christopher Faylor
2005-05-11 19:05 ` Eli Zaretskii
2005-05-11 19:58 ` Christopher Faylor
2005-05-11 23:16 ` Eli Zaretskii
2005-05-11 6:56 ` PATCH: Use getche on Win32 Mark Mitchell
2005-05-11 13:42 ` Eli Zaretskii
2005-05-13 11:42 ` Mark Mitchell [this message]
2005-05-13 15:18 ` Eli Zaretskii
2005-05-11 7:07 ` Eli Zaretskii
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=42844946.4090202@codesourcery.com \
--to=mark@codesourcery.com \
--cc=bug-readline@gnu.org \
--cc=drow@false.org \
--cc=eliz@gnu.org \
--cc=gdb-patches@sources.redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox