all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* bug#16852: 24.3.50; Elisp indentation bug in `indent-for-tab-command'
@ 2014-02-23 14:20 Oleh
  2014-02-23 15:02 ` Dmitry Gutov
  0 siblings, 1 reply; 4+ messages in thread
From: Oleh @ 2014-02-23 14:20 UTC (permalink / raw
  To: 16852

Hi,

With emacs -Q, and this text in *scratch*:

(
 progn
 )

and the point anywhere on the last line, pressing "C-i" will produce this
indentation:

(
 progn
  )

So one extra space is being inserted before the closing paren.

regards,
Oleh






^ permalink raw reply	[flat|nested] 4+ messages in thread

* bug#16852: 24.3.50; Elisp indentation bug in `indent-for-tab-command'
  2014-02-23 14:20 bug#16852: 24.3.50; Elisp indentation bug in `indent-for-tab-command' Oleh
@ 2014-02-23 15:02 ` Dmitry Gutov
       [not found]   ` <CAJ1SQWAA5e1f-BaKvBvCz24FUzJYVvjqgzrVpGLHe-Yi-=n98g@mail.gmail.com>
  0 siblings, 1 reply; 4+ messages in thread
From: Dmitry Gutov @ 2014-02-23 15:02 UTC (permalink / raw
  To: Oleh; +Cc: 16852

Oleh <o.krehel@tue.nl> writes:

> With emacs -Q, and this text in *scratch*:
>
> (
>  progn
>  )
>
> and the point anywhere on the last line, pressing "C-i" will produce this
> indentation:
>
> (
>  progn
>   )
>
> So one extra space is being inserted before the closing paren.

Looks intentional to me. `progn' usually has some arguments. Add at
least one and see that the closing paren lines up to it just fine.





^ permalink raw reply	[flat|nested] 4+ messages in thread

* bug#16852: 24.3.50; Elisp indentation bug in `indent-for-tab-command'
       [not found]   ` <CAJ1SQWAA5e1f-BaKvBvCz24FUzJYVvjqgzrVpGLHe-Yi-=n98g@mail.gmail.com>
@ 2014-02-23 16:10     ` Dmitry Gutov
  2014-02-24 20:52       ` Stefan Monnier
  0 siblings, 1 reply; 4+ messages in thread
From: Dmitry Gutov @ 2014-02-23 16:10 UTC (permalink / raw
  To: Oleh Krehel; +Cc: 16852

On 23.02.2014 17:37, Oleh Krehel wrote:
> It's not consistent with `indent-sexp'. Call it from beginning of sexp and see
> that it reverts to the first state, which is the correct one in my opinion.

Hmm. Looks like a bug in `indent-sexp' to me.

(Please keep the bug address in Cc).





^ permalink raw reply	[flat|nested] 4+ messages in thread

* bug#16852: 24.3.50; Elisp indentation bug in `indent-for-tab-command'
  2014-02-23 16:10     ` Dmitry Gutov
@ 2014-02-24 20:52       ` Stefan Monnier
  0 siblings, 0 replies; 4+ messages in thread
From: Stefan Monnier @ 2014-02-24 20:52 UTC (permalink / raw
  To: Dmitry Gutov; +Cc: Oleh Krehel, 16852

> Hmm. Looks like a bug in `indent-sexp' to me.

Indeed, there are some discrepancies in Elisp between indent-region and
indent-line.  We should fix them by scrapping the indent-region code,
I think (tho this requires improving the generic indent-region code so
that it can also do the "indent-comment" thingy that Elisp's
indent-region does).


        Stefan





^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2014-02-24 20:52 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-02-23 14:20 bug#16852: 24.3.50; Elisp indentation bug in `indent-for-tab-command' Oleh
2014-02-23 15:02 ` Dmitry Gutov
     [not found]   ` <CAJ1SQWAA5e1f-BaKvBvCz24FUzJYVvjqgzrVpGLHe-Yi-=n98g@mail.gmail.com>
2014-02-23 16:10     ` Dmitry Gutov
2014-02-24 20:52       ` Stefan Monnier

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.