[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#63790: 30.0.50; prog-fill-reindent-defun regression
From: |
Juri Linkov |
Subject: |
bug#63790: 30.0.50; prog-fill-reindent-defun regression |
Date: |
Thu, 08 Jun 2023 19:59:53 +0300 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/30.0.50 (x86_64-pc-linux-gnu) |
>> (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.
I still don't understand why 'M-q' now does the same what 'C-M-q' was
doing all the time with code indentation? Also why 'prog-fill-reindent-defun'
can't indent the region of code, but only the region of comments?
Shouldn't 'M-q' only refill comments, and 'C-M-q' only indent code, as before?