[Emacs-diffs] /srv/bzr/emacs/trunk r109007: Remove "#define unix" that is no longer needed (Bug#11905).

[Paul Eggert]

------------------------------------------------------------
revno: 109007
fixes bug: http://debbugs.gnu.org/11905
committer: Paul Eggert <address@hidden>
branch nick: trunk
timestamp: Tue 2012-07-10 17:01:21 -0700
message:
  Remove "#define unix" that is no longer needed (Bug#11905).
  
  Merge from gnulib to make "#define unix" unnecessary, incorporating:
  2012-07-10 getloadavg: clean out old Emacs and Autoconf cruft
  2012-07-09 getopt: Simplify after Emacs changed.
  * src/s/aix4-2.h (unix): Remove; no longer needed.
modified:
  ChangeLog
  lib/getloadavg.c
  m4/getloadavg.m4
  m4/getopt.m4
  src/ChangeLog
  src/s/aix4-2.h

=== modified file 'ChangeLog'
--- a/ChangeLog 2012-07-10 07:37:17 +0000
+++ b/ChangeLog 2012-07-11 00:01:21 +0000
@@ -1,3 +1,10 @@
+2012-07-10  Paul Eggert  <address@hidden>
+
+       Remove "#define unix" that is no longer needed (Bug#11905).
+       Merge from gnulib to make "#define unix" unnecessary, incorporating:
+       2012-07-10 getloadavg: clean out old Emacs and Autoconf cruft
+       2012-07-09 getopt: Simplify after Emacs changed.
+
 2012-07-10  Glenn Morris  <address@hidden>
 
        * configure.ac (DATA_START, DATA_SEG_BITS, PENDING_OUTPUT_COUNT):

=== modified file 'lib/getloadavg.c'
--- a/lib/getloadavg.c  2012-07-09 08:34:39 +0000
+++ b/lib/getloadavg.c  2012-07-11 00:01:21 +0000
@@ -80,45 +80,23 @@
    We also #define LDAV_PRIVILEGED if a program will require
    special installation to be able to call getloadavg.  */
 
-/* "configure" defines CONFIGURING_GETLOADAVG to sidestep problems
-   with partially-configured source directories.  */
-
-#ifndef CONFIGURING_GETLOADAVG
-# include <config.h>
-# include <stdbool.h>
-#endif
+#include <config.h>
 
 /* Specification.  */
 #include <stdlib.h>
 
 #include <errno.h>
+#include <stdbool.h>
 #include <stdio.h>
 
 # include <sys/types.h>
 
-/* Both the Emacs and non-Emacs sections want this.  Some
-   configuration files' definitions for the LOAD_AVE_CVT macro (like
-   sparc.h's) use macros like FSCALE, defined here.  */
-# if defined (unix) || defined (__unix)
+# if HAVE_SYS_PARAM_H
 #  include <sys/param.h>
 # endif
 
 # include "intprops.h"
 
-/* The existing Emacs configuration files define a macro called
-   LOAD_AVE_CVT, which accepts a value of type LOAD_AVE_TYPE, and
-   returns the load average multiplied by 100.  What we actually want
-   is a macro called LDAV_CVT, which returns the load average as an
-   unmultiplied double.
-
-   For backwards compatibility, we'll define LDAV_CVT in terms of
-   LOAD_AVE_CVT, but future machine config files should just define
-   LDAV_CVT directly.  */
-
-# if !defined (LDAV_CVT) && defined (LOAD_AVE_CVT)
-#  define LDAV_CVT(n) (LOAD_AVE_CVT (n) / 100.0)
-# endif
-
 # if !defined (BSD) && defined (ultrix)
 /* Ultrix behaves like BSD on Vaxen.  */
 #  define BSD

=== modified file 'm4/getloadavg.m4'
--- a/m4/getloadavg.m4  2012-06-22 17:20:00 +0000
+++ b/m4/getloadavg.m4  2012-07-11 00:01:21 +0000
@@ -105,6 +105,8 @@
 [
 # Figure out what our getloadavg.c needs.
 
+AC_CHECK_HEADERS_ONCE([sys/param.h])
+
 # On HPUX9, an unprivileged user can get load averages this way.
 if test $gl_func_getloadavg_done = no; then
   AC_CHECK_FUNCS([pstat_getdynamic], [gl_func_getloadavg_done=yes])

=== modified file 'm4/getopt.m4'
--- a/m4/getopt.m4      2012-07-09 08:34:39 +0000
+++ b/m4/getopt.m4      2012-07-11 00:01:21 +0000
@@ -1,4 +1,4 @@
-# getopt.m4 serial 43
+# getopt.m4 serial 44
 dnl Copyright (C) 2002-2006, 2008-2012 Free Software Foundation, Inc.
 dnl This file is free software; the Free Software Foundation
 dnl gives unlimited permission to copy and/or distribute it,
@@ -9,18 +9,17 @@
 [
   m4_divert_text([DEFAULTS], [gl_getopt_required=POSIX])
   AC_REQUIRE([gl_UNISTD_H_DEFAULTS])
+  AC_REQUIRE([gl_GETOPT_CHECK_HEADERS])
   dnl Other modules can request the gnulib implementation of the getopt
   dnl functions unconditionally, by defining gl_REPLACE_GETOPT_ALWAYS.
   dnl argp.m4 does this.
   m4_ifdef([gl_REPLACE_GETOPT_ALWAYS], [
-    gl_GETOPT_IFELSE([], [])
     REPLACE_GETOPT=1
   ], [
     REPLACE_GETOPT=0
-    gl_GETOPT_IFELSE([
+    if test -n "$gl_replace_getopt"; then
       REPLACE_GETOPT=1
-    ],
-    [])
+    fi
   ])
   if test $REPLACE_GETOPT = 1; then
     dnl Arrange for getopt.h to be created.
@@ -38,12 +37,6 @@
   AC_REQUIRE([gl_FUNC_GETOPT_POSIX])
 ])
 
-AC_DEFUN([gl_GETOPT_IFELSE],
-[
-  AC_REQUIRE([gl_GETOPT_CHECK_HEADERS])
-  AS_IF([test -n "$gl_replace_getopt"], [$1], [$2])
-])
-
 # Determine whether to replace the entire getopt facility.
 AC_DEFUN([gl_GETOPT_CHECK_HEADERS],
 [

=== modified file 'src/ChangeLog'
--- a/src/ChangeLog     2012-07-10 23:24:36 +0000
+++ b/src/ChangeLog     2012-07-11 00:01:21 +0000
@@ -1,5 +1,8 @@
 2012-07-10  Paul Eggert  <address@hidden>
 
+       Remove "#define unix" that is no longer needed (Bug#11905).
+       * s/aix4-2.h (unix): Remove; no longer needed.
+
        EMACS_TIME simplification (Bug#11875).
        This replaces macros (which typically do not work in GDB)
        with functions, typedefs and enums, making the code easier to debug.

=== modified file 'src/s/aix4-2.h'
--- a/src/s/aix4-2.h    2012-06-11 23:17:11 +0000
+++ b/src/s/aix4-2.h    2012-07-11 00:01:21 +0000
@@ -40,9 +40,6 @@
 
 /* Special items needed to make Emacs run on this system.  */
 
-/* AIX doesn't define this.  */
-#define unix 1
-
 /* Perry Smith <address@hidden> says these are correct.  */
 #define SIGNALS_VIA_CHARACTERS
 #define CLASH_DETECTION