* lisp.h (DEFUN): Remove bogus use of sizeof (struct Lisp_Subr).

This didn't break anything, but it didn't help either.
It's confusing to put a bogus integer in a place where the actual
value does not matter.

diff --git a/src/ChangeLog b/src/ChangeLog
index aa0a0a39bb8a5e029ca5beff2c7c65bbde8c8fa9..1b4edb91d1abba6b33a28409b5e3b51006f04d5c 100644 (file)
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,3 +1,10 @@
+2011-06-19  Paul Eggert  <eggert@cs.ucla.edu>
+
+       * lisp.h (DEFUN): Remove bogus use of sizeof (struct Lisp_Subr).
+       This didn't break anything, but it didn't help either.
+       It's confusing to put a bogus integer in a place where the actual
+       value does not matter.
+
 2011-06-18  Paul Eggert  <eggert@cs.ucla.edu>
 
        * lisp.h (union Lisp_Object.i): EMACS_INT, not EMACS_UINT.

diff --git a/src/lisp.h b/src/lisp.h
index 2dcf73b75061ef5a7567e0761933238e5101b4d9..e065bad10d04dc0f5a3d0ad3201e2cc341633ec3 100644 (file)
--- a/src/lisp.h
+++ b/src/lisp.h
@@ -1890,7 +1890,7 @@ typedef struct {
 #define DEFUN(lname, fnname, sname, minargs, maxargs, intspec, doc) \
   Lisp_Object fnname DEFUN_ARGS_ ## maxargs ;                          \
   static DECL_ALIGN (struct Lisp_Subr, sname) =                                \
-    { PVEC_SUBR | (sizeof (struct Lisp_Subr) / sizeof (EMACS_INT)),    \
+    { PVEC_SUBR,                                                       \
       { .a ## maxargs = fnname },                              \
       minargs, maxargs, lname, intspec, 0};                            \
   Lisp_Object fnname