* config/sparc/xm-sun4os4.h: Define MEM_FNS_DECLARED and include

	<memory.h>.
	Include <malloc.h> rather than declaring malloc functions ourself.

	* ser-unix.c (set_tty_state): Don't ignore errors setting process
	group.
	* inflow.c (terminal_inferior): If attach_flag set, ignore errors
	from set_tty_state.

3 files changed, 31 insertions, 17 deletions

diff --git a/gdb/ChangeLog b/gdb/ChangeLog
index 7ad66db..b2fd729 100644
--- a/gdb/ChangeLog
+++ b/gdb/ChangeLog
@@ -1,3 +1,19 @@
+Tue Jul 27 12:07:38 1993  Jim Kingdon  (kingdon@lioth.cygnus.com)
+
+	* config/sparc/xm-sun4os4.h: Define MEM_FNS_DECLARED and include
+	<memory.h>.
+	Include <malloc.h> rather than declaring malloc functions ourself.
+
+	* ser-unix.c (set_tty_state): Don't ignore errors setting process
+	group.
+	* inflow.c (terminal_inferior): If attach_flag set, ignore errors
+	from set_tty_state.
+
+	* fork-child.c (fork_inferior): Only quote exec file if needed.
+
+	* mipsread.c (parse_symbol): Remove 21 Jul 93 change with
+	stTypedef inside an stBlock.
+
 Tue Jul 27 12:36:49 1993  Peter Schauer  (pes@regent.e-technik.tu-muenchen.de)
 
 	* breakpoint.c (breakpoint_1):  Walk the breakpoint chain to decide if
diff --git a/gdb/config/sparc/xm-sun4os4.h b/gdb/config/sparc/xm-sun4os4.h
index 03d1c8f..1f74de3 100644
--- a/gdb/config/sparc/xm-sun4os4.h
+++ b/gdb/config/sparc/xm-sun4os4.h
@@ -41,18 +41,20 @@ Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.  */
 #define MMAP_BASE_ADDRESS	0xE0000000	/* First mapping here */
 #define MMAP_INCREMENT		0x01000000	/* Increment to next mapping */
 
-#ifndef __STDC__
-/* GCC (2.3-ish at least) fixes this in fixincludes.  */
-/* /usr/include/malloc.h is included by vx-share/xdr_ld.  */
-/* /usr/include/malloc.h defines these w/o prototypes (and uses
-   char * instead of void *).  */
+/* /usr/include/malloc.h is included by vx-share/xdr_ld, and might
+   declare these using char * not void *.  The following should work with
+   acc, gcc, or /bin/cc.  */
+
 #define MALLOC_INCOMPATIBLE
-extern char* malloc ();
-extern char* realloc ();
-/* Yes, it really does define it as returning int, both in malloc.h and
-   stdlib.h, at least on SunOS 4.1.1.  */
-extern int free ();
-#endif
+#include <malloc.h>
+
+/* acc for SunOS4 comes with string.h and memory.h headers which we
+   pick up somewhere (where?) and which use char *, not void *.  The
+   following should work with acc, gcc, or /bin/cc, at least with
+   SunOS 4.1.1.  */
+
+#define MEM_FNS_DECLARED
+#include <memory.h>
 
 /* SunOS 4.x uses nonstandard "char *" as type of third argument to ptrace() */
 
diff --git a/gdb/ser-unix.c b/gdb/ser-unix.c
index b147c73..5f07ade 100644
--- a/gdb/ser-unix.c
+++ b/gdb/ser-unix.c
@@ -159,9 +159,7 @@ set_tty_state(scb, state)
   if (!job_control)
     return 0;
 
-  /* Need to ignore errors, at least if attach_flag is set.  */
-  tcsetpgrp (scb->fd, state->process_group);
-  return 0;
+  return tcsetpgrp (scb->fd, state->process_group);
 #endif
 
 #ifdef HAVE_TERMIO
@@ -177,9 +175,7 @@ set_tty_state(scb, state)
   if (!job_control)
     return 0;
 
-  /* Need to ignore errors, at least if attach_flag is set.  */
-  ioctl (scb->fd, TIOCSPGRP, &state->process_group);
-  return 0;
+  return ioctl (scb->fd, TIOCSPGRP, &state->process_group);
 #endif
 }