[Emacs-diffs] master d090be1: Change inhibit-point-motion-hooks to t

[Stefan Monnier]

branch: master
commit d090be146176e9acee89fdaadc86e2eb26209ef5
Author: Stefan Monnier <address@hidden>
Commit: Stefan Monnier <address@hidden>

    Change inhibit-point-motion-hooks to t
    
    * src/textprop.c (syms_of_textprop): Default Vinhibit_point_motion_hooks
    to t and document it as obsolete.
---
 etc/NEWS       |    2 ++
 src/textprop.c |   12 ++++++++++--
 2 files changed, 12 insertions(+), 2 deletions(-)

diff --git a/etc/NEWS b/etc/NEWS
index 24f6d58..4333efb 100644
--- a/etc/NEWS
+++ b/etc/NEWS
@@ -756,6 +756,8 @@ a typographically-correct documents.
 
 * Incompatible Lisp Changes in Emacs 25.1
 
+** `inhibit-point-motion-hooks' now defaults to t and is obsolete.
+
 ** The optional `predicate' argument of `lisp-complete-symbol' no longer
 has any effect.  (This change was made in Emacs 24.4 but was not
 advertised at the time.)
diff --git a/src/textprop.c b/src/textprop.c
index 0a591d0..96d88ed 100644
--- a/src/textprop.c
+++ b/src/textprop.c
@@ -2344,8 +2344,16 @@ returned. */);
 
   DEFVAR_LISP ("inhibit-point-motion-hooks", Vinhibit_point_motion_hooks,
               doc: /* If non-nil, don't run `point-left' and `point-entered' 
text properties.
-This also inhibits the use of the `intangible' text property.  */);
-  Vinhibit_point_motion_hooks = Qnil;
+This also inhibits the use of the `intangible' text property.
+
+This variable is obsolete since Emacs-25.1.  Use `cursor-intangible-mode'
+or `cursor-sensor-mode' instead.  */);
+  /* FIXME: We should make-obsolete-variable, but that signals too many
+     warnings in code which does (let ((inhibit-point-motion-hooks t)) ...)
+     Ideally, make-obsolete-variable should let us specify that only the nil
+     value is obsolete, but that requires too many changes in bytecomp.el,
+     so for now we'll keep it "obsolete via the docstring".  */
+  Vinhibit_point_motion_hooks = Qt;
 
   DEFVAR_LISP ("text-property-default-nonsticky",
               Vtext_property_default_nonsticky,