GNU bug report logs - #67462
30.0.50; prog-fill-reindent-defun does not respect buffer-local fill-paragraph-function

Previous Next

Full log

View this message in rfc822 format

From: Jens Schmidt <jschmidt4gnu <at> vodafonemail.de>
To: Eli Zaretskii <eliz <at> gnu.org>,  dmitry <at> gutov.dev
Cc: 67462 <at> debbugs.gnu.org
Subject: bug#67462: 30.0.50; prog-fill-reindent-defun does not respect buffer-local fill-paragraph-function
Date: Thu, 30 Nov 2023 21:40:19 +0100

Eli Zaretskii <eliz <at> gnu.org> writes:

>> From: Jens Schmidt <jschmidt4gnu <at> vodafonemail.de>
>> Cc: 67462 <at> debbugs.gnu.org,  dmitry <at> gutov.dev
>> Date: Wed, 29 Nov 2023 22:28:49 +0100
>>
>> No, indent-region does a good job and there is nothing to fix here.  But
>> starting with Emacs 30 it does that job (also) on M-q, where previously
>> other functionality may have been bound.
>>
>> To see the difference, please try executing in Emacs 29 vs master on
>> Emacs's own, configured Makefile:
>>
>>   ./src/emacs -Q Makefile --eval '(search-forward "SUBDIR_MAKEFILES_IN")'
>>   M-q
>
> So the problem is that the new code does its job, just with different
> results than the mode-specific fill-paragraph-function did in Emacs 29
> and before?

Yes.

> If so, perhaps we should offer a user option to let users choose which
> method to use, because some might prefer the new behavior?

I'd be fine with that, and also with making the new behavior the
default.  A reference to that user option would also nicely complete the
NEWS entry, which IMO currently is a bit weak in mentioning the possible
consequences of the new feature.

Dmitry, what do you think?

Previous Next

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