GNU bug report logs - #26187
Bug in indent-sexp

Previous Next

Full log

Message #16 received at control <at> debbugs.gnu.org (full text, mbox):

From: npostavs <at> users.sourceforge.net
To: Gdobbins <gdobbins <at> protonmail.com>
Cc: 26187 <at> debbugs.gnu.org
Subject: Re: bug#26187: Bug in indent-sexp
Date: Mon, 20 Mar 2017 23:22:00 -0400

tags 26187 patch
quit

Gdobbins <gdobbins <at> protonmail.com> writes:

> Sure. In the form:
> 
> (defun lisp-indent-defform (state _indent-point)
>   (goto-char (car (cdr state)))
>   (forward-line 1)
>   (if (> (point) (car (cdr (cdr state))))|
>       (progn
>         (goto-char (car (cdr state)))
>         (+ lisp-body-indent (current-column)))))
> 
> where | is point, call indent-sexp and the result will be
> 
> (defun lisp-indent-defform (state _indent-point)
>   (goto-char (car (cdr state)))
>   (forward-line 1)
>   (if (> (point) (car (cdr (cdr state))))|
> (progn
>   (goto-char (car (cdr state)))
>   (+ lisp-body-indent (current-column)))))
> 
> which is incorrect.

Hmm, the original behaviour is that no indentation happens at all which
seems kind of odd to me.  But that's better than breaking things I
guess.  Have you done copyright assignment for Emacs, or should I mark
your patch as a tiny change?

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.