diff src/process-unix.c @ 563:183866b06e0b

[xemacs-hg @ 2001-05-24 07:50:48 by ben] Makefile.in.in, abbrev.c, alloc.c, buffer.c, bytecode.c, callint.c, callproc.c, casetab.c, chartab.c, cmdloop.c, cmds.c, console-msw.c, console-msw.h, console-stream.c, console-tty.c, console-x.c, console.c, data.c, database.c, debug.c, device-gtk.c, device-msw.c, device-tty.c, device-x.c, device.c, dialog-gtk.c, dialog-msw.c, dialog-x.c, dialog.c, dired-msw.c, dired.c, doc.c, doprnt.c, dragdrop.c, editfns.c, eldap.c, eldap.h, elhash.c, emacs-widget-accessors.c, emacs.c, emodules.c, esd.c, eval.c, event-Xt.c, event-gtk.c, event-msw.c, event-stream.c, events.c, extents.c, faces.c, file-coding.c, fileio.c, filelock.c, floatfns.c, fns.c, font-lock.c, frame-gtk.c, frame-x.c, frame.c, general-slots.h, glade.c, glyphs-gtk.c, glyphs-msw.c, glyphs-widget.c, glyphs-x.c, glyphs.c, glyphs.h, gpmevent.c, gui-gtk.c, gui-x.c, gui.c, gutter.c, hpplay.c, indent.c, input-method-xlib.c, insdel.c, intl.c, keymap.c, libsst.c, libsst.h, linuxplay.c, lisp.h, lread.c, lstream.c, lstream.h, macros.c, marker.c, md5.c, menubar-gtk.c, menubar-msw.c, menubar-x.c, menubar.c, minibuf.c, miscplay.c, miscplay.h, mule-ccl.c, mule-charset.c, mule-wnnfns.c, mule.c, nas.c, ntplay.c, ntproc.c, objects-gtk.c, objects-msw.c, objects-x.c, objects.c, postgresql.c, print.c, process-nt.c, process-unix.c, process.c, ralloc.c, rangetab.c, redisplay.c, scrollbar.c, search.c, select-gtk.c, select-x.c, select.c, sgiplay.c, sheap.c, sound.c, specifier.c, sunplay.c, symbols.c, symeval.h, symsinit.h, syntax.c, sysdep.c, toolbar-msw.c, toolbar.c, tooltalk.c, ui-byhand.c, ui-gtk.c, undo.c, unexaix.c, unexapollo.c, unexconvex.c, unexec.c, widget.c, win32.c, window.c: -- defsymbol -> DEFSYMBOL. -- add an error type to all errors. -- eliminate the error functions in eval.c that let you just use Qerror as the type. -- redo the error API to be more consistent, sensibly named, and easier to use. -- redo the error hierarchy somewhat. create new errors: structure-formation-error, gui-error, invalid-constant, stack-overflow, out-of-memory, process-error, network-error, sound-error, printing-unreadable-object, base64-conversion- error; coding-system-error renamed to text-conversion error; some others. -- fix Mule problems in error strings in emodules.c, tooltalk.c. -- fix error handling in mswin open-network-stream. -- Mule-ize all sound files and clean up the headers. -- nativesound.h -> sound.h and used for all sound files. -- move some shared stuff into glyphs-shared.c: first attempt at eliminating some of the massive GTK code duplication. xemacs.mak: add glyphs-shared.c. xemacs-faq.texi: document how to debug X errors subr.el: fix doc string to reflect reality
author ben
date Thu, 24 May 2001 07:51:33 +0000
parents ed498ef2108b
children 6db80f4ab17c
line wrap: on
line diff
--- a/src/process-unix.c	Thu May 24 06:30:21 2001 +0000
+++ b/src/process-unix.c	Thu May 24 07:51:33 2001 +0000
@@ -479,8 +479,8 @@
       numeric_addr = inet_addr ((char *) XSTRING_DATA (host));
       if (NUMERIC_ADDR_ERROR)
 	{
-	  maybe_error (Qprocess, errb,
-		       "Unknown host \"%s\"", XSTRING_DATA (host));
+	  maybe_signal_error (Qio_error, "Unknown host", host,
+				   Qprocess, errb);
 	  return 0;
 	}
 
@@ -1074,7 +1074,7 @@
       int save_errno = errno;
       close_descriptor_pair (forkin, forkout);
       errno = save_errno;
-      report_file_error ("Doing fork", Qnil);
+      report_process_error ("Doing fork", Qunbound);
     }
 
   /* #### dmoore - why is this commented out, otherwise we leave
@@ -1104,7 +1104,7 @@
     close_descriptor_pair (forkin, forkout);
     close_descriptor_pair (inchannel, outchannel);
     errno = save_errno;
-    report_file_error ("Opening pty or pipe", Qnil);
+    report_process_error ("Opening pty or pipe", Qunbound);
     return 0; /* not reached */
   }
 }
@@ -1293,11 +1293,15 @@
 	     the unwritten data. */
 	  writeret = Lstream_write (XLSTREAM (DATA_OUTSTREAM(p)), chunkbuf,
 				    chunklen);
-	  signal (SIGPIPE, old_sigpipe);
-	  if (writeret < 0)
-	    /* This is a real error.  Blocking errors are handled
-	       specially inside of the filedesc stream. */
-	    report_file_error ("writing to process", list1 (proc));
+	  {
+	    int save_errno = errno;
+	    signal (SIGPIPE, old_sigpipe);
+	    errno = save_errno;
+	    if (writeret < 0)
+	      /* This is a real error.  Blocking errors are handled
+		 specially inside of the filedesc stream. */
+	      report_process_error ("writing to process", proc);
+	  }
 	  while (Lstream_was_blocked_p (XLSTREAM (p->pipe_outstream)))
 	    {
 	      /* Buffer is full.  Wait, accepting input;
@@ -1541,8 +1545,9 @@
 #ifdef ESRCH
       if (errno != ESRCH)
 #endif
-	error ("kill (%ld, %ld) failed: %s",
-	       (long) pgid, (long) signo, strerror (errno));
+	signal_ferror_with_frob (Qio_error, lisp_strerror (errno),
+				 "kill (pgid=%ld, signo=%ld) failed",
+				 (long) pgid, (long) signo);
     }
 }
 
@@ -1586,11 +1591,11 @@
   retval = getaddrinfo (ext_host, NULL, &hints, &res);
   if (retval != 0)
     {
-      char *gai_error;
+      Bufbyte *gai_error;
 
       EXTERNAL_TO_C_STRING (gai_strerror (retval), gai_error, Qnative);
-      maybe_error (Qprocess, ERROR_ME_NOT,
-		   "%s \"%s\"", gai_error, XSTRING_DATA (host));
+      maybe_signal_error (Qio_error, gai_error, host,
+			       Qprocess, ERROR_ME_NOT);
       canonname = host;
     }
   else
@@ -1641,7 +1646,7 @@
   CHECK_STRING (host);
 
   if (!EQ (protocol, Qtcp) && !EQ (protocol, Qudp))
-    invalid_argument ("Unsupported protocol", protocol);
+    invalid_constant ("Unsupported protocol", protocol);
 
   {
 #ifdef USE_GETADDRINFO
@@ -1680,10 +1685,10 @@
     retval = getaddrinfo (ext_host, portstring, &hints, &res);
     if (retval != 0)
       {
-	char *gai_error;
+	Bufbyte *gai_error;
 
 	EXTERNAL_TO_C_STRING (gai_strerror (retval), gai_error, Qnative);
-	error ("%s/%s %s", XSTRING_DATA (host), portstring, gai_error);
+	signal_error (Qio_error, gai_error, list2 (host, service));
       }
 
     /* address loop */
@@ -1848,9 +1853,10 @@
 	errno = xerrno;
 
 	if (failed_connect)
-	  report_file_error ("connection failed", list2 (host, name));
+	  report_network_error ("connection failed", list3 (Qunbound, host,
+							    name));
 	else
-	  report_file_error ("error creating socket", list1 (name));
+	  report_network_error ("error creating socket", name);
       }
   }
 
@@ -1858,8 +1864,10 @@
   outch = dup (s);
   if (outch < 0)
     {
+      int save_errno = errno;
       close (s); /* this used to be leaked; from Kyle Jones */
-      report_file_error ("error duplicating socket", list1 (name));
+      errno = save_errno;
+      report_network_error ("error duplicating socket", name);
     }
 
   set_socket_nonblocking_maybe (inch, port, "tcp");
@@ -1913,15 +1921,17 @@
   thettl = (unsigned char) XINT (ttl);
 
   if ((udp = getprotobyname ("udp")) == NULL)
-    type_error (Qinvalid_operation, "No info available for UDP protocol");
+    invalid_operation ("No info available for UDP protocol", Qunbound);
 
   /* Init the sockets. Yes, I need 2 sockets. I couldn't duplicate one. */
   if ((rs = socket (PF_INET, SOCK_DGRAM, udp->p_proto)) < 0)
-    report_file_error ("error creating socket", list1(name));
+    report_network_error ("error creating socket", name);
   if ((ws = socket (PF_INET, SOCK_DGRAM, udp->p_proto)) < 0)
     {
+      int save_errno = errno;
       close (rs);
-      report_file_error ("error creating socket", list1(name));
+      errno = save_errno;
+      report_network_error ("error creating socket", name);
     }
 
   /* This will be used for both sockets */
@@ -1942,9 +1952,12 @@
   /* bind socket name */
   if (bind (rs, (struct sockaddr *)&sa, sizeof(sa)))
     {
+      int save_errno = errno;
       close (rs);
       close (ws);
-      report_file_error ("error binding socket", list2(name, port));
+      errno = save_errno;
+      report_network_error ("error binding socket", list3 (Qunbound, name,
+							   port));
     }
 
   /* join multicast group */
@@ -1953,9 +1966,12 @@
   if (setsockopt (rs, IPPROTO_IP, IP_ADD_MEMBERSHIP,
 		  &imr, sizeof (struct ip_mreq)) < 0)
     {
+      int save_errno = errno;
       close (ws);
       close (rs);
-      report_file_error ("error adding membership", list2(name, dest));
+      errno = save_errno;
+      report_network_error ("error adding membership", list3 (Qunbound, name,
+							      dest));
     }
 
   /* Socket configuration for writing ----------------------- */
@@ -2018,7 +2034,8 @@
 #endif
 
       errno = xerrno;
-      report_file_error ("error connecting socket", list2(name, port));
+      report_network_error ("error connecting socket", list3 (Qunbound, name,
+							      port));
     }
 
 #ifdef CONNECT_NEEDS_SLOWED_INTERRUPTS
@@ -2029,9 +2046,11 @@
   if (setsockopt (ws, IPPROTO_IP, IP_MULTICAST_TTL,
 		  &thettl, sizeof (thettl)) < 0)
     {
+      int save_errno = errno;
       close (rs);
       close (ws);
-      report_file_error ("error setting ttl", list2(name, ttl));
+      errno = save_errno;
+      report_network_error ("error setting ttl", list3 (Qunbound, name, ttl));
     }
 
   set_socket_nonblocking_maybe (rs, theport, "udp");