Fix calls to buffer modification hooks from replace-buffer-contents

* src/editfns.c (Freplace_buffer_contents): Don't call buffer
modification hooks if nothing was deleted/inserted.  (Bug#32237)

diff --git a/src/editfns.c b/src/editfns.c
index d1a6bfbbb1bd25c28257e510f6a0b8663c484566..cf596aec37c04319b9ded5e99a8c55128e58c6aa 100644 (file)
--- a/src/editfns.c
+++ b/src/editfns.c
@@ -3254,10 +3254,12 @@ differences between the two buffers.  */)
       from = BEGV + k;
 
       /* Find the last character position to be changed.  */
-      for (l = size_a; l > 0 && !bit_is_set (ctx.deletions, l - 1); l--)
+      for (l = size_a; l > k && !bit_is_set (ctx.deletions, l - 1); l--)
        ;
       to = BEGV + l;
-      prepare_to_modify_buffer (from, to, NULL);
+      /* If k >= l, it means nothing needs to be deleted.  */
+      if (k < l)
+       prepare_to_modify_buffer (from, to, NULL);
       specbind (Qinhibit_modification_hooks, Qt);
       modification_hooks_inhibited = true;
     }
@@ -3308,11 +3310,16 @@ differences between the two buffers.  */)
   SAFE_FREE ();
   rbc_quitcounter = 0;
 
-  if (modification_hooks_inhibited)
+  if (modification_hooks_inhibited && from <= to)
     {
       ptrdiff_t updated_to = to + ZV - BEGV - size_a;
-      signal_after_change (from, to - from, updated_to - from);
-      update_compositions (from, updated_to, CHECK_INSIDE);
+      /* Only call after-change-functions if something was actually
+        inserted.  */
+      if (from < updated_to)
+       {
+         signal_after_change (from, to - from, updated_to - from);
+         update_compositions (from, updated_to, CHECK_INSIDE);
+       }
     }
 
   return Qnil;