From a77a2f50060cb771dbf06674b52559f90bf9d38a Mon Sep 17 00:00:00 2001 From: Taylor R Campbell Date: Sat, 26 Nov 2022 10:01:50 +0000 Subject: Change some comments to xfail. --- NEWS | 1 + paredit.el | 78 +++++++++++++++++++++++++++++++++++++++++++++++++++++--------- test.el | 56 ++++++++++++++++++++++++-------------------- 3 files changed, 99 insertions(+), 36 deletions(-) diff --git a/NEWS b/NEWS index 21f86dc..a4ab2bc 100644 --- a/NEWS +++ b/NEWS @@ -10,6 +10,7 @@ Current development version: https://paredit.org/paredit-beta.el ** Version 26 (beta) *** M-s (paredit-splice-sexp) now restores column in text fields like ielm. +*** Deletion now respects `delete-active-region'. ** Version 25 -- 2022-11-25 diff --git a/paredit.el b/paredit.el index 9140b39..4f8c7ab 100644 --- a/paredit.el +++ b/paredit.el @@ -340,7 +340,7 @@ Paredit behaves badly if parentheses are unbalanced, so exercise ("C-j" paredit-C-j) "Deleting & Killing" - (("C-d" ,@paredit-forward-delete-keys) + (,paredit-forward-delete-keys paredit-forward-delete ("(quu|x \"zot\")" "(quu| \"zot\")") ("(quux |\"zot\")" @@ -356,6 +356,13 @@ Paredit behaves badly if parentheses are unbalanced, so exercise "(\"zo|\" quux)") ("(foo (|) bar)" "(foo | bar)") ("(foo bar)|" "(foo bar|)")) + ("C-d" paredit-delete-char + ("(quu|x \"zot\")" "(quu| \"zot\")") + ("(quux |\"zot\")" + "(quux \"|zot\")" + "(quux \"|ot\")") + ("(foo (|) bar)" "(foo | bar)") + ("|(foo bar)" "(|foo bar)")) ("C-k" paredit-kill ("(foo bar)| ; Useless comment!" "(foo bar)|") @@ -1241,6 +1248,33 @@ This is expected to be called only in `paredit-comment-dwim'; do not ;;;; Character Deletion +(defun paredit-delete-char (&optional argument) + "Delete a character forward or move forward over a delimiter. +If on an opening S-expression delimiter, move forward into the + S-expression. +If on a closing S-expression delimiter, refuse to delete unless the + S-expression is empty, in which case delete the whole S-expression. +With a numeric prefix argument N, delete N characters forward. +With a `C-u' prefix argument, simply delete a character forward, + without regard for delimiter balancing. + +Like `delete-char', ignores `delete-active-region'." + (interactive "P") + (let ((delete-active-region nil)) + (paredit-forward-delete argument))) + +(defun paredit-delete-active-region-p () + "True if the region is active and to be deleted." + (and (paredit-region-active-p) + (boundp 'delete-active-region) + (eq delete-active-region t))) + +(defun paredit-kill-active-region-p () + "True if the region is active and to be killed." + (and (paredit-region-active-p) + (boundp 'delete-active-region) + (eq delete-active-region 'kill))) + (defun paredit-forward-delete (&optional argument) "Delete a character forward or move forward over a delimiter. If on an opening S-expression delimiter, move forward into the @@ -1249,16 +1283,27 @@ If on a closing S-expression delimiter, refuse to delete unless the S-expression is empty, in which case delete the whole S-expression. With a numeric prefix argument N, delete N characters forward. With a `C-u' prefix argument, simply delete a character forward, - without regard for delimiter balancing." + without regard for delimiter balancing. + +If `delete-active-region' is enabled and the mark is active and + no prefix argument is specified, act as `paredit-delete-region' + or `paredit-kill-region' as appropriate instead." (interactive "P") - (cond ((or (consp argument) (eobp)) + (cond ((consp argument) (delete-char +1)) ((integerp argument) - (if (< argument 0) - (paredit-backward-delete argument) + (let ((delete-active-region nil)) + (if (< argument 0) + (paredit-backward-delete argument) (while (> argument 0) (paredit-forward-delete) - (setq argument (- argument 1))))) + (setq argument (- argument 1)))))) + ((paredit-delete-active-region-p) + (paredit-delete-region (region-beginning) (region-end))) + ((paredit-kill-active-region-p) + (paredit-kill-region (region-beginning) (region-end))) + ((eobp) + (delete-char +1)) ((paredit-in-string-p) (paredit-forward-delete-in-string)) ((paredit-in-comment-p) @@ -1348,17 +1393,28 @@ If on an opening S-expression delimiter, refuse to delete unless the S-expression is empty, in which case delete the whole S-expression. With a numeric prefix argument N, delete N characters backward. With a `C-u' prefix argument, simply delete a character backward, - without regard for delimiter balancing." + without regard for delimiter balancing. + +If `delete-active-region' is enabled and the mark is active and + no prefix argument is specified, act as `paredit-delete-region' + or `paredit-kill-region' as appropriate instead." (interactive "P") - (cond ((or (consp argument) (bobp)) + (cond ((consp argument) ;++ Should this untabify? (delete-char -1)) ((integerp argument) - (if (< argument 0) - (paredit-forward-delete (- 0 argument)) + (let ((delete-active-region nil)) + (if (< argument 0) + (paredit-forward-delete (- 0 argument)) (while (> argument 0) (paredit-backward-delete) - (setq argument (- argument 1))))) + (setq argument (- argument 1)))))) + ((paredit-delete-active-region-p) + (paredit-delete-region (region-beginning) (region-end))) + ((paredit-kill-active-region-p) + (paredit-kill-region (region-beginning) (region-end))) + ((bobp) + (delete-char -1)) ((paredit-in-string-p) (paredit-backward-delete-in-string)) ((paredit-in-comment-p) diff --git a/test.el b/test.el index 77850fa..c0aabb2 100644 --- a/test.el +++ b/test.el @@ -345,24 +345,34 @@ Four arguments: the paredit command, the text of the buffer ("(|" "|" error) (")|" "|" error))) -(dolist (command '(paredit-delete-region paredit-kill-region)) +(paredit-test 'paredit-delete-region '(("|foo" error))) +(paredit-test 'paredit-kill-region '(("|foo" error))) + +(let ((deletion-tests + '(("|foo_" "|") + ("|(foo)_" "|") + (";;; f|oo (bar ;_baz\n(zot)\n" ";;; f|baz\n(zot)\n") + ("(foo |bar_ baz)\n" "(foo | baz)\n") + ("(foo |(bar \"baz\" ; quux\n zot)\n _mumble)" + "(foo |mumble)") + ("(foo (bar |baz) (quux _zot) mumble)" "(foo (bar |zot) mumble)") + ("(foo bar ;baz| quux\n zot_)" error) + ("(foo bar ;baz| quux\n _zot\n mumble)" + "(foo bar ;baz|zot\n mumble)") + ("(foo bar| baz ;quux (_)\n zot)" error) + ("(foo bar| baz ;quux ()_\n zot)" + "(foo bar|\n zot)")))) + (paredit-test 'paredit-delete-region deletion-tests) ;++ Need to check whether `paredit-kill-region' updates the kill ring ;++ correctly. - (paredit-test command - '(("|foo" error) - ("|foo_" "|") - ("|(foo)_" "|") - (";;; f|oo (bar ;_baz\n(zot)\n" ";;; f|baz\n(zot)\n") - ("(foo |bar_ baz)\n" "(foo | baz)\n") - ("(foo |(bar \"baz\" ; quux\n zot)\n _mumble)" - "(foo |mumble)") - ("(foo (bar |baz) (quux _zot) mumble)" "(foo (bar |zot) mumble)") - ("(foo bar ;baz| quux\n zot_)" error) - ("(foo bar ;baz| quux\n _zot\n mumble)" - "(foo bar ;baz|zot\n mumble)") - ("(foo bar| baz ;quux (_)\n zot)" error) - ("(foo bar| baz ;quux ()_\n zot)" - "(foo bar|\n zot)")))) + (paredit-test 'paredit-kill-region deletion-tests) + (if (boundp 'delete-active-region) + (let ((delete-active-region t) + (transient-mark-mode t) + (mark-active t)) + ;; XXX check that paredit-delete-char is not affected + (paredit-test 'paredit-forward-delete deletion-tests) + (paredit-test 'paredit-backward-delete deletion-tests)))) ;;; The hairiest paredit command: paredit-kill. @@ -1630,16 +1640,12 @@ Four arguments: the paredit command, the text of the buffer (let ((backward-cases '(("(hello \"world\")|" "(hello \"|\")" - "(|\"\")" - ;; error or nop -- XXX broken - ) + "(|\"\")") + (xfail "(|\"\")" "(|\"\")") ("(hello \"|world\")" - "(|\"world\")" - ;; error or nop -- XXX broken - ) - ("(|hello \"world\")" - ;; error or nop -- XXX broken - ) + "(|\"world\")") + (xfail "(|\"world\")" "(|\"world\")") + (xfail "(|hello \"world\")" "(|hello \"world\")") ("|(hello \"world\")" "|(hello \"world\")")))) (paredit-test 'paredit-backward-kill-word backward-cases) (let ((current-prefix-arg -1)) -- cgit v1.2.1