[Emacs-diffs] /srv/bzr/emacs/trunk r111566: * lisp.h (eabs): Define unconditionally.

[Paul Eggert]

------------------------------------------------------------
revno: 111566
fixes bug: http://debbugs.gnu.org/13419
committer: Paul Eggert <address@hidden>
branch nick: trunk
timestamp: Sat 2013-01-19 14:33:29 -0800
message:
  * lisp.h (eabs): Define unconditionally.
  
  The old "#if !defined (eabs)" was an unnecessary revenant of back
  when this macro was called "abs".  Document 'eabs' better.
modified:
  src/ChangeLog
  src/lisp.h

=== modified file 'src/ChangeLog'
--- a/src/ChangeLog     2013-01-19 20:38:13 +0000
+++ b/src/ChangeLog     2013-01-19 22:33:29 +0000
@@ -1,3 +1,9 @@
+2013-01-19  Paul Eggert  <address@hidden>
+
+       * lisp.h (eabs): Define unconditionally (Bug#13419).
+       The old "#if !defined (eabs)" was an unnecessary revenant of back
+       when this macro was called "abs".  Document 'eabs' better.
+
 2013-01-19  Glenn Morris  <address@hidden>
 
        * fns.c (Frandom): Doc fix.

=== modified file 'src/lisp.h'
--- a/src/lisp.h        2013-01-19 04:44:34 +0000
+++ b/src/lisp.h        2013-01-19 22:33:29 +0000
@@ -3711,12 +3711,11 @@
 /* Set up the name of the machine we're running on.  */
 extern void init_system_name (void);
 
-/* We used to use `abs', but that clashes with system headers on some
-   platforms, and using a name reserved by Standard C is a bad idea
-   anyway.  */
-#if !defined (eabs)
+/* Return the absolute value of X.  X should be a signed integer
+   expression without side effects, and X's absolute value should not
+   exceed the maximum for its promoted type.  This is called 'eabs'
+   because 'abs' is reserved by the C standard.  */
 #define eabs(x)         ((x) < 0 ? -(x) : (x))
-#endif
 
 /* Return a fixnum or float, depending on whether VAL fits in a Lisp
    fixnum.  */