|
From: | Dmitry Gutov |
Subject: | bug#63790: 30.0.50; prog-fill-reindent-defun regression |
Date: | Fri, 9 Jun 2023 04:58:35 +0300 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.11.0 |
On 08/06/2023 19:59, Juri Linkov wrote:
(re-search-forward "\\s-*\\s<" (line-end-position) t)It's looking for a comment that begins after point (possibly preceded by whitespace). There is no comment after point in the presented scenario.It's nil in the reported case, so 'fill-paragraph' is not called.I guess when there is an active region, we would force the behavior to "refill" the region, no matter whether it is inside a comment, or contains a comment, or outside of any comments and simply contains code?While 'prog-fill-reindent-defun' doesn't support indentation of an arbitrary region of code and indents only the top-level list (defun), it looks like the right thing is to fill the region.
Okay?
I still don't understand why 'M-q' now does the same what 'C-M-q' was doing all the time with code indentation?
C-M-q (indent-pp-sexp) reindents the list that follows point. Not the same thing. And it's only available in Lisp.
> Also why 'prog-fill-reindent-defun' > can't indent the region of code, but only the region of comments? Do you want it to?
Shouldn't 'M-q' only refill comments, and 'C-M-q' only indent code, as before?
Up until now, we thought that making two actions on one key binding available is a good thing, given that the context usually helps to disambiguate. This one seems like an exception, but IMHO not a strong enough one to roll back the change.
[Prev in Thread] | Current Thread | [Next in Thread] |