Skip to content

Fix cl-assertion-failed in diff-hl-revert-hunk caused by track-changes#287

Open
jamescherti wants to merge 1 commit into
dgutov:masterfrom
jamescherti:bug-revert-hunk-cl-assertion-failed
Open

Fix cl-assertion-failed in diff-hl-revert-hunk caused by track-changes#287
jamescherti wants to merge 1 commit into
dgutov:masterfrom
jamescherti:bug-revert-hunk-cl-assertion-failed

Conversation

@jamescherti
Copy link
Copy Markdown
Contributor

@jamescherti jamescherti commented Jun 7, 2026

This pull request fixes #286

It disables track-changes-mode inside the temporary diff-hl-revert buffer. This prevents asynchronous background timers from firing while execution is suspended at the yes-or-no-p prompt.

Because the buffer is narrowed to highlight the hunk, the tracker previously attempted to access coordinates outside the visible region, causing an assertion failure.

@jamescherti jamescherti force-pushed the bug-revert-hunk-cl-assertion-failed branch from 7120fbc to 376606a Compare June 7, 2026 12:24
@jamescherti jamescherti closed this Jun 7, 2026
@jamescherti jamescherti force-pushed the bug-revert-hunk-cl-assertion-failed branch from 376606a to 81b7628 Compare June 7, 2026 12:28
@jamescherti jamescherti reopened this Jun 7, 2026
@jamescherti jamescherti force-pushed the bug-revert-hunk-cl-assertion-failed branch 2 times, most recently from 5a8e149 to c73cab3 Compare June 7, 2026 12:37
It disables track-changes-mode inside the temporary diff-hl-revert
buffer. This prevents asynchronous background timers from firing while
execution is suspended at the yes-or-no-p prompt.

Because the buffer is narrowed to highlight the hunk, the tracker
previously attempted to access coordinates outside the visible region,
causing an assertion failure.
@jamescherti jamescherti force-pushed the bug-revert-hunk-cl-assertion-failed branch from c73cab3 to ac1b266 Compare June 7, 2026 21:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

cl-assertion-failed in diff-hl-revert-hunk due to track-changes in Emacs 30+

1 participant