Re: regression: filling comments in C++ code (today's CVS)

[Martin Stjernholm]

"Stefan Monnier" <monnier+gnu/address@hidden> wrote:

> Ah, so it's because filladapt does not faithfully implement the
> fill-paragraph interface.

Well, it's not that Kyle E Jones just was being stupid when he wrote
filladapt. He can't do it any better way since there's no hook to
replace fill-context-prefix which implements adaptive-fill-mode.

It'd be nice if fill.el provided a function that only did the pure
text filling between two positions, preferably also with the option to
keep a fill postfix or to fill inside a rectangle. Doing the actual
filling is pretty messy, so other packages rather device various
kludges to avoid duplicating that code.

> For the sake of correctness in cases other than filladapt, how about
> setting fill-paragraph-function (while still keeping the rebinding
> of M-q) so that calling fill-paragraph DTRT.

That wouldn't work when filladapt is used, but it's perhaps good
enough for your purposes. A more thorough solution would be to put an
advice around fill-paragraph.