From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 24805 invoked by alias); 28 Apr 2005 20:45:48 -0000 Mailing-List: contact gdb-patches-help@sources.redhat.com; run by ezmlm Precedence: bulk List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sources.redhat.com Received: (qmail 24783 invoked from network); 28 Apr 2005 20:45:44 -0000 Received: from unknown (HELO mail.codesourcery.com) (65.74.133.9) by sourceware.org with SMTP; 28 Apr 2005 20:45:44 -0000 Received: (qmail 28705 invoked from network); 28 Apr 2005 20:45:43 -0000 Received: from localhost (HELO ?10.253.201.43?) (mitchell@127.0.0.1) by mail.codesourcery.com with SMTP; 28 Apr 2005 20:45:43 -0000 Message-ID: <42714B6F.1030009@codesourcery.com> Date: Thu, 28 Apr 2005 20:45:00 -0000 From: Mark Mitchell User-Agent: Mozilla Thunderbird 1.0.2 (Windows/20050317) MIME-Version: 1.0 To: Daniel Jacobowitz CC: Eli Zaretskii , bug-readline@gnu.org, gdb-patches@sources.redhat.com Subject: Re: PATCH: Readline on MinGW References: <4270886B.1020806@codesourcery.com> <01c54c2d$Blat.v2.4$f3029840@zahav.net.il> <20050428201215.GA14846@nevyn.them.org> In-Reply-To: <20050428201215.GA14846@nevyn.them.org> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit X-SW-Source: 2005-04/txt/msg00399.txt.bz2 Daniel Jacobowitz wrote: > On Thu, Apr 28, 2005 at 11:06:45PM +0300, Eli Zaretskii wrote: > >>I am not sure `raise' is a 100% compatible replacement for `kill', >>since the latter really delivers a signal, and so is subject to rules >>regarding blocked signals, while `raise' simlpy calls the signal >>handler and AFAIK is not specified to observe blocked signals. >> >>So I suggest to use `raise' only on systems, such as MinGW, which lack >>`kill', not everywhere. > > > Could you give me a reference for this? POSIX disagrees: > > http://www.opengroup.org/onlinepubs/009695399/functions/raise.html Yes, I looked at that (and the GNU/Linux, and Solaris manual pages) before making that change. But, I'm happy to change the code to check for "kill", and use "raise" only if unavailable, as Eli suggests, if that's necessary. Eli, just to be clear, you said you "support" the patches I posted (with this exception). I'm interpreting that as "I hope they go into upstream readline", rather than as "it's OK to check these patches into the GDB repository." If you actually meant the latter, let me know. :-) -- Mark Mitchell CodeSourcery, LLC mark@codesourcery.com (916) 791-8304