]> git.eshelyaron.com Git - emacs.git/commitdiff
Add named defun for transpose-sexps-default-function (bug#60654)
authorTheodor Thornhill <theo@thornhill.no>
Mon, 9 Jan 2023 06:52:38 +0000 (07:52 +0100)
committerYuan Fu <casouri@gmail.com>
Mon, 9 Jan 2023 20:33:56 +0000 (12:33 -0800)
* lisp/simple.el (transpose-sexps-default-function): Move the lambda
into its own function.
(transpose-sexps-function): Refer to it by name.
* etc/NEWS: Mention the change.

etc/NEWS
lisp/simple.el

index 60dab575da65ccbdbb0d92789fe828ea32dc491e..3aa8f2abb77ff5e9420c5115acfddad5d926be2c 100644 (file)
--- a/etc/NEWS
+++ b/etc/NEWS
@@ -57,6 +57,10 @@ trash when deleting.  Default is nil.
 Emacs now can set this variable to customize the behavior of the
 'transpose-sexps' function.
 
+** New function 'transpose-sexps-default-function'.
+The previous implementation is moved into its own function, to be
+bound by transpose-sexps-function'.
+
 ** New function 'treesit-transpose-sexps'.
 treesit.el now unconditionally sets 'transpose-sexps-function' for all
 Tree-sitter modes.  This functionality utilizes the new
index 690968ca938e75670089a91f02480553c3bbb0b4..c8c5542caeecf415049295bde7f845120c9b8938 100644 (file)
@@ -8436,37 +8436,39 @@ are interchanged."
   (interactive "*p")
   (transpose-subr 'forward-word arg))
 
-(defvar transpose-sexps-function
-  (lambda (arg)
-    ;; Here we should try to simulate the behavior of
-    ;; (cons (progn (forward-sexp x) (point))
-    ;;       (progn (forward-sexp (- x)) (point)))
-    ;; Except that we don't want to rely on the second forward-sexp
-    ;; putting us back to where we want to be, since forward-sexp-function
-    ;; might do funny things like infix-precedence.
-    (if (if (> arg 0)
-           (looking-at "\\sw\\|\\s_")
-         (and (not (bobp))
-              (save-excursion
-                 (forward-char -1)
-                 (looking-at "\\sw\\|\\s_"))))
-        ;; Jumping over a symbol.  We might be inside it, mind you.
-       (progn (funcall (if (> arg 0)
-                           #'skip-syntax-backward #'skip-syntax-forward)
-                       "w_")
-              (cons (save-excursion (forward-sexp arg) (point)) (point)))
-      ;; Otherwise, we're between sexps.  Take a step back before jumping
-      ;; to make sure we'll obey the same precedence no matter which
-      ;; direction we're going.
-      (funcall (if (> arg 0) #'skip-syntax-backward #'skip-syntax-forward)
-               " .")
-      (cons (save-excursion (forward-sexp arg) (point))
-           (progn (while (or (forward-comment (if (> arg 0) 1 -1))
-                             (not (zerop (funcall (if (> arg 0)
-                                                      #'skip-syntax-forward
-                                                    #'skip-syntax-backward)
-                                                  ".")))))
-                  (point)))))
+(defun transpose-sexps-default-function (arg)
+  "Default method to locate a pair of points for transpose-sexps."
+  ;; Here we should try to simulate the behavior of
+  ;; (cons (progn (forward-sexp x) (point))
+  ;;       (progn (forward-sexp (- x)) (point)))
+  ;; Except that we don't want to rely on the second forward-sexp
+  ;; putting us back to where we want to be, since forward-sexp-function
+  ;; might do funny things like infix-precedence.
+  (if (if (> arg 0)
+         (looking-at "\\sw\\|\\s_")
+       (and (not (bobp))
+            (save-excursion
+               (forward-char -1)
+               (looking-at "\\sw\\|\\s_"))))
+      ;; Jumping over a symbol.  We might be inside it, mind you.
+      (progn (funcall (if (> arg 0)
+                         #'skip-syntax-backward #'skip-syntax-forward)
+                     "w_")
+            (cons (save-excursion (forward-sexp arg) (point)) (point)))
+    ;; Otherwise, we're between sexps.  Take a step back before jumping
+    ;; to make sure we'll obey the same precedence no matter which
+    ;; direction we're going.
+    (funcall (if (> arg 0) #'skip-syntax-backward #'skip-syntax-forward)
+             " .")
+    (cons (save-excursion (forward-sexp arg) (point))
+         (progn (while (or (forward-comment (if (> arg 0) 1 -1))
+                           (not (zerop (funcall (if (> arg 0)
+                                                    #'skip-syntax-forward
+                                                  #'skip-syntax-backward)
+                                                ".")))))
+                (point)))))
+
+(defvar transpose-sexps-function #'transpose-sexps-default-function
   "If non-nil, `transpose-sexps' delegates to this function.
 
 This function takes one argument ARG, a number.  Its expected