Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
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


  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