From 38899558cf242187966d8bf1771a3c8e48eb5090 Mon Sep 17 00:00:00 2001 From: Taylor R Campbell Date: Wed, 5 Apr 2017 18:47:53 +0000 Subject: Don't reindent single-line edits in paredit-raise-sexp. Add a couple trivial tests for paredit-raise-sexp. Bug reported by Alex Plotnick. --- paredit.el | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) (limited to 'paredit.el') diff --git a/paredit.el b/paredit.el index 3b69058..c54997d 100644 --- a/paredit.el +++ b/paredit.el @@ -2156,7 +2156,15 @@ If the point is on an S-expression, such as a string or a symbol, not (delete-region (point) (scan-sexps (point) 1)) (let* ((indent-start (point)) (indent-end (save-excursion (insert sexps) (point)))) - (indent-region indent-start indent-end nil))))) + ;; If the expression spans multiple lines, its indentation is + ;; probably broken, so reindent it -- but don't reindent + ;; anything that we didn't touch outside the expression. + ;; + ;; XXX What if the *column* of the starting point was preserved + ;; too? Should we avoid reindenting in that case? + (if (not (eq (save-excursion (goto-char indent-start) (point-at-eol)) + (save-excursion (goto-char indent-end) (point-at-eol)))) + (indent-region indent-start indent-end nil)))))) ;;; The effects of convolution on the surrounding whitespace are pretty ;;; random. If you have better suggestions, please let me know. -- cgit v1.2.1