[Emacs-diffs] /srv/bzr/emacs/trunk r108940: * configure.in (BROKEN_SA_RESTART): Doc fix.

[Glenn Morris]

------------------------------------------------------------
revno: 108940
committer: Glenn Morris <address@hidden>
branch nick: trunk
timestamp: Sat 2012-07-07 11:16:15 -0700
message:
  * configure.in (BROKEN_SA_RESTART): Doc fix.
  
  * src/sysdep.c, src/s/irix6-5.h: Related comments.
modified:
  ChangeLog
  configure.in
  src/s/irix6-5.h
  src/sysdep.c

=== modified file 'ChangeLog'
--- a/ChangeLog 2012-07-07 01:03:46 +0000
+++ b/ChangeLog 2012-07-07 18:16:15 +0000
@@ -1,5 +1,7 @@
 2012-07-07  Glenn Morris  <address@hidden>
 
+       * configure.in (BROKEN_SA_RESTART): Doc fix.
+
        * configure.in: Rather than checking for things then undef'ing
        them on some platforms, simply don't check for them.
        (getwd): Don't check for it on unixware.

=== modified file 'configure.in'
--- a/configure.in      2012-07-07 01:03:46 +0000
+++ b/configure.in      2012-07-07 18:16:15 +0000
@@ -3140,9 +3140,11 @@
    gnu-kfreebsd) opsysfile="s/gnu-linux.h" ;;
 
    hpux11)
-     dnl SA_RESTART resets the timeout of `select', so don't use it.
-     AC_DEFINE(BROKEN_SA_RESTART, 1, [Define if SA_RESTART should not
-       be used.])
+     dnl See comments in sysdep.c:sys_signal.
+     dnl SA_RESTART resets the timeout of `select' on hpux11.
+     dnl Defining BROKEN_SA_RESTART is not the same as undef'ing SA_RESTART.
+     AC_DEFINE(BROKEN_SA_RESTART, 1, [Define if SA_RESTART should only
+       be used in batch mode.])
      dnl It works to open the pty's tty in the parent (Emacs), then
      dnl close and reopen it in the child.
      AC_DEFINE(USG_SUBTTY_WORKS, 1, [Define for USG systems where it

=== modified file 'src/s/irix6-5.h'
--- a/src/s/irix6-5.h   2012-06-11 23:17:11 +0000
+++ b/src/s/irix6-5.h   2012-07-07 18:16:15 +0000
@@ -80,7 +80,7 @@
 
 #define NARROWPROTO 1
 
-#undef SA_RESTART
+#undef SA_RESTART     /* not the same as defining BROKEN_SA_RESTART */
 
 #undef TIOCSIGSEND             /* defined in usg5-4-common.h */
 

=== modified file 'src/sysdep.c'
--- a/src/sysdep.c      2012-07-07 03:06:00 +0000
+++ b/src/sysdep.c      2012-07-07 18:16:15 +0000
@@ -1484,9 +1484,10 @@
 #if defined (SA_RESTART)
   /* Emacs mostly works better with restartable system services. If this
      flag exists, we probably want to turn it on here.
-     However, on some systems this resets the timeout of `select'
-     which means that `select' never finishes if it keeps getting signals.
-     BROKEN_SA_RESTART is defined on those systems.  */
+     However, on some systems (only hpux11 at present) this resets the
+     timeout of `select' which means that `select' never finishes if
+     it keeps getting signals.
+     We define BROKEN_SA_RESTART on those systems.  */
   /* It's not clear why the comment above says "mostly works better".  --Stef
      When SYNC_INPUT is set, we don't want SA_RESTART because we need to poll
      for pending input so we need long-running syscalls to be interrupted