The Wayback Machine - https://web.archive.org/web/20210324144208/https://github.com/emacs-evil/evil/pull/1430
Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix bugs in evil-with-undo' and evil-undo-pop' with undo-tree. #1430

Open
wants to merge 1 commit into
base: master
from

Conversation

@tsc25
Copy link

@tsc25 tsc25 commented Mar 6, 2021

Addresses #1074

  • evil-with-undo:
    nconc'ing onto front of buffer-undo-list here can corrupt buffer-undo-list
    when in undo-tree-mode in rare circumstances (see issue #1074). Leave
    standard undo machinery to work as usual when undo is enabled. Deal with
    disabled undo by temporarily enabling then disabling undo, and transferring
    any undo changes to evil-temporary-undo.

  • evil-undo-pop:
    This function called `undo' directly from Elisp, which is wrong when in
    undo-tree-mode. Fix this by calling undo-tree-undo instead when in
    undo-tree-mode.

Addresses #1074

- evil-with-undo:
  nconc'ing onto front of buffer-undo-list here can corrupt buffer-undo-list
  when in undo-tree-mode in rare circumstances (see issue #1074). Leave
  standard undo machinery to work as usual when undo is enabled. Deal with
  disabled undo by temporarily enabling then disabling undo, and transferring
  any undo changes to evil-temporary-undo.

- evil-undo-pop:
  This function called `undo' directly from Elisp, which is wrong when in
  undo-tree-mode. Fix this by calling undo-tree-undo instead when in
  undo-tree-mode.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant