diff options
| author | Stefan Monnier <monnier@iro.umontreal.ca> | 2023-09-13 18:49:55 -0400 |
|---|---|---|
| committer | Stefan Monnier <monnier@iro.umontreal.ca> | 2023-09-13 18:49:55 -0400 |
| commit | 0065621d0d36112922792d7c46085230e999cba3 (patch) | |
| tree | c6dc5182920d8475120a53ac3ee630c860b7115f /src | |
| parent | 6cc6455e931913e42ef844c4a7d6ddcd0bf53788 (diff) | |
| download | emacs-0065621d0d36112922792d7c46085230e999cba3.tar.gz emacs-0065621d0d36112922792d7c46085230e999cba3.tar.bz2 emacs-0065621d0d36112922792d7c46085230e999cba3.zip | |
(report_overlay_modification): Fix bug#65929
Somehow the new overlay code in Emacs-29 changed slightly
the test to decide when to run the `modification-hooks` of overlays,
with the consequence that these hook functions end up being executed
also when text is deleted right after an empty overlay, which is
contrary to Emacs-28 behavior as well as contrary to the Texinfo doc.
* src/buffer.c (report_overlay_modification): Better reproduce the
Emacs-28 code.
* test/src/buffer-tests.el (overlay-modification-hooks):
Add corresponding test.
Diffstat (limited to 'src')
| -rw-r--r-- | src/buffer.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/buffer.c b/src/buffer.c index 252231357bc..db362ffe7b0 100644 --- a/src/buffer.c +++ b/src/buffer.c @@ -4091,7 +4091,7 @@ report_overlay_modification (Lisp_Object start, Lisp_Object end, bool after, } /* Test for intersecting intervals. This does the right thing for both insertion and deletion. */ - if (! insertion || (end_arg > obegin && begin_arg < oend)) + if (end_arg > obegin && begin_arg < oend) { Lisp_Object prop = Foverlay_get (overlay, Qmodification_hooks); if (!NILP (prop)) |