Fix indent-sexp of #s(...) (Bug#31984)

* lisp/emacs-lisp/lisp-mode.el (indent-sexp): Look for a sexp that
ends after the current line.
* test/lisp/emacs-lisp/lisp-mode-tests.el (indent-sexp-go): New test.
This commit is contained in:
Noam Postavsky 2018-07-19 06:40:54 -04:00
parent 59e8533286
commit 1b4b96597c
2 changed files with 20 additions and 2 deletions

View file

@ -1199,8 +1199,14 @@ ENDPOS is encountered."
(setq endpos (copy-marker (setq endpos (copy-marker
(if endpos endpos (if endpos endpos
;; Get error now if we don't have a complete sexp ;; Get error now if we don't have a complete sexp
;; after point. ;; after point. We actually look for a sexp which
(save-excursion (forward-sexp 1) (point))))) ;; ends after the current line so that we properly
;; indent things like #s(...). This might not be
;; needed if Bug#15998 is fixed.
(let ((eol (line-end-position)))
(save-excursion (while (and (< (point) eol) (not (eobp)))
(forward-sexp 1))
(point))))))
(save-excursion (save-excursion
(while (let ((indent (lisp-indent-calc-next parse-state)) (while (let ((indent (lisp-indent-calc-next parse-state))
(ppss (lisp-indent-state-ppss parse-state))) (ppss (lisp-indent-state-ppss parse-state)))

View file

@ -113,6 +113,18 @@ noindent\" 3
;; we're indenting ends on the previous line. ;; we're indenting ends on the previous line.
(should (equal (buffer-string) original))))) (should (equal (buffer-string) original)))))
(ert-deftest indent-sexp-go ()
"Make sure `indent-sexp' doesn't stop after #s."
;; See https://debbugs.gnu.org/cgi/bugreport.cgi?bug=31984.
(with-temp-buffer
(emacs-lisp-mode)
(insert "#s(foo\nbar)\n")
(goto-char (point-min))
(indent-sexp)
(should (equal (buffer-string) "\
#s(foo
bar)\n"))))
(ert-deftest lisp-indent-region () (ert-deftest lisp-indent-region ()
"Test basics of `lisp-indent-region'." "Test basics of `lisp-indent-region'."
(with-temp-buffer (with-temp-buffer