From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 12309 invoked by alias); 9 Aug 2004 06:48:28 -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 12284 invoked from network); 9 Aug 2004 06:48:27 -0000 Received: from unknown (HELO takamaka.act-europe.fr) (142.179.108.108) by sourceware.org with SMTP; 9 Aug 2004 06:48:27 -0000 Received: by takamaka.act-europe.fr (Postfix, from userid 507) id E602F47D91; Sun, 8 Aug 2004 23:48:26 -0700 (PDT) Date: Mon, 09 Aug 2004 06:48:00 -0000 From: Joel Brobecker To: Peter Schauer , Kevin Buettner Cc: gdb-patches@sources.redhat.com Subject: Re: [patch/rfa] xm-aix4.h cleanup Message-ID: <20040809064826.GO1192@gnat.com> References: <20040809063644.GN1192@gnat.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="XuV1QlJbYrcVoo+x" Content-Disposition: inline In-Reply-To: <20040809063644.GN1192@gnat.com> User-Agent: Mutt/1.4i X-SW-Source: 2004-08/txt/msg00295.txt.bz2 --XuV1QlJbYrcVoo+x Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-length: 697 > This leaves: > > . FIVE_ARG_PTRACE: Will become OBE thanks to a patch from MarkK. > > . USE_O_NOCTTY: I think the idea for that one is to always use O_NOCTTY, > regardless of the system. So the macro becomes moot. Will send a patch. > > . #pragma alloca: Need to continue the discussion. Actually, I didn't look closely enough, even after Mark's answer. This is already dealt with, so we can remove this one as well. So I'm suggesting this patch instead (which such scraps the alloca #pragma in addition to what the previous patch was already removing). Is this OK? (the only little detail is that I am not sure I can verify this works or not. I'll see what I can do). -- Joel --XuV1QlJbYrcVoo+x Content-Type: text/plain; charset=us-ascii Content-Disposition: attachment; filename="xm-aix.diff" Content-length: 2330 Index: xm-aix4.h =================================================================== RCS file: /cvs/src/src/gdb/config/xm-aix4.h,v retrieving revision 1.6 diff -u -p -r1.6 xm-aix4.h --- xm-aix4.h 2 Dec 2001 02:57:13 -0000 1.6 +++ xm-aix4.h 9 Aug 2004 06:46:44 -0000 @@ -20,27 +20,6 @@ Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA. */ -/* The following text is taken from config/rs6000.mh: - * # The IBM version of /usr/include/rpc/rpc.h has a bug -- it says - * # `extern fd_set svc_fdset;' without ever defining the type fd_set. - * # Unfortunately this occurs in the vx-share code, which is not configured - * # like the rest of GDB (e.g. it doesn't include "defs.h"). - * # We circumvent this bug by #define-ing fd_set here, but undefining it in - * # the xm-rs6000.h file before ordinary modules try to use it. FIXME, IBM! - * MH_CFLAGS='-Dfd_set=int' - * So, here we do the undefine...which has to occur before we include - * below. - */ -#undef fd_set - -#include - -/* At least as of AIX 3.2, we have termios. */ -#define HAVE_TERMIOS 1 -/* #define HAVE_TERMIO 1 */ - -#define USG 1 - #define FIVE_ARG_PTRACE /* This system requires that we open a terminal with O_NOCTTY for it to @@ -48,44 +27,3 @@ #define USE_O_NOCTTY -/* Brain death inherited from PC's pervades. */ -#undef NULL -#define NULL 0 - -/* The IBM compiler requires this in order to properly compile alloca(). */ -#pragma alloca - -/* There is no vfork. */ - -#define vfork fork - -char *termdef (); - -/* Signal handler for SIGWINCH `window size changed'. */ - -#define SIGWINCH_HANDLER aix_resizewindow -extern void aix_resizewindow (int); - -/* `lines_per_page' and `chars_per_line' are local to utils.c. Rectify this. */ - -#define SIGWINCH_HANDLER_BODY \ - \ -/* Respond to SIGWINCH `window size changed' signal, and reset GDB's \ - window settings appropriately. */ \ - \ -void \ -aix_resizewindow (signo) \ - int signo; \ -{ \ - int fd = fileno (stdout); \ - if (isatty (fd)) { \ - int val; \ - \ - val = atoi (termdef (fd, 'l')); \ - if (val > 0) \ - lines_per_page = val; \ - val = atoi (termdef (fd, 'c')); \ - if (val > 0) \ - chars_per_line = val; \ - } \ -} --XuV1QlJbYrcVoo+x--