[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Emacs-diffs] master 5959b48: Default CHECK_LISP_OBJECT_TYPE to "no"
From: |
Paul Eggert |
Subject: |
[Emacs-diffs] master 5959b48: Default CHECK_LISP_OBJECT_TYPE to "no" |
Date: |
Sun, 17 Dec 2017 21:44:32 -0500 (EST) |
branch: master
commit 5959b48ece0abe4639667c023da6363859088676
Author: Paul Eggert <address@hidden>
Commit: Paul Eggert <address@hidden>
Default CHECK_LISP_OBJECT_TYPE to "no"
* configure.ac: Go back to not defining CHECK_LISP_OBJECT_TYPE by
default for developer builds, since it is no longer that useful.
We can make it a no-op entirely later, if in practice it's not
that helpful to enable it.
---
configure.ac | 7 +++----
etc/NEWS | 7 +++++++
2 files changed, 10 insertions(+), 4 deletions(-)
diff --git a/configure.ac b/configure.ac
index 562b19a..ec1418b 100644
--- a/configure.ac
+++ b/configure.ac
@@ -900,10 +900,9 @@ AC_ARG_ENABLE([gcc-warnings],
AC_ARG_ENABLE([check-lisp-object-type],
[AS_HELP_STRING([--enable-check-lisp-object-type],
- [Enable compile-time checks for the Lisp_Object data type,
- which can catch some bugs during development.
- The default is "no" if --enable-gcc-warnings is "no".])])
-if test "${enable_check_lisp_object_type-$gl_gcc_warnings}" != "no"; then
+ [Enable compile time checks for the Lisp_Object data type,
+ which can catch some bugs during development.])])
+if test "$enable_check_lisp_object_type" = yes; then
AC_DEFINE([CHECK_LISP_OBJECT_TYPE], 1,
[Define to enable compile-time checks for the Lisp_Object data type.])
fi
diff --git a/etc/NEWS b/etc/NEWS
index 1382f96..1ab1930 100644
--- a/etc/NEWS
+++ b/etc/NEWS
@@ -38,6 +38,13 @@ interpreter or modules that it uses. If your platform
supports it you
can enable it when configuring, e.g., './configure CFLAGS="-g3 -O2
-mmpx -fcheck-pointer-bounds"' on Intel MPX platforms.
+** Emacs now normally uses a pointer type instead of an integer type
+for the fundamental word in the Emacs Lisp interpreter, to help
+catch typos and support -fcheck-pointer-bounds. The 'configure'
+option --enable-check-lisp-object-type is therefore no longer as
+useful and so is no longer enabled by default in developer builds,
+to reduce differences between developer and production builds.
+
* Startup Changes in Emacs 27.1
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- [Emacs-diffs] master 5959b48: Default CHECK_LISP_OBJECT_TYPE to "no",
Paul Eggert <=