From: Paul Eggert Date: Sat, 19 Jan 2013 22:33:29 +0000 (-0800) Subject: * lisp.h (eabs): Define unconditionally. X-Git-Tag: emacs-24.3.90~173^2~7^2~256 X-Git-Url: http://git.eshelyaron.com/gitweb/?a=commitdiff_plain;h=d7a0a7c9bddcf0b5005c392d5f00702d26cc5b9c;p=emacs.git * 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. Fixes: debbugs:13419 --- diff --git a/src/ChangeLog b/src/ChangeLog index eb393c93c4a..1383b12ed68 100644 --- a/src/ChangeLog +++ b/src/ChangeLog @@ -1,3 +1,9 @@ +2013-01-19 Paul Eggert + + * 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 * fns.c (Frandom): Doc fix. diff --git a/src/lisp.h b/src/lisp.h index 48fba653203..faf4b1b8edd 100644 --- a/src/lisp.h +++ b/src/lisp.h @@ -3711,12 +3711,11 @@ extern char *egetenv (const char *); /* 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. */