GNU bug report logs - #78561
[PATCH] Add semantic linefeed support for paragraph filling

Previous Next

Package: emacs;

Reported by: Roi Martin <jroi.martin <at> gmail.com>

Date: Fri, 23 May 2025 09:59:02 UTC

Severity: normal

Tags: patch

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

Bug is archived. No further changes may be made.

Full log


Message #76 received at 78561 <at> debbugs.gnu.org (full text, mbox):

From: Roi Martin <jroi.martin <at> gmail.com>
To: Eli Zaretskii <eliz <at> gnu.org>, Juri Linkov <juri <at> linkov.net>
Cc: 78561 <at> debbugs.gnu.org
Subject: Re: bug#78561: [PATCH] Add semantic linefeed support for paragraph
 filling
Date: Sun, 15 Jun 2025 11:12:49 +0200
Eli Zaretskii <eliz <at> gnu.org> writes:

>> (setq-local fill-paragraph-function 'fill-paragraph-semlf)
>> 
>> or this
>> 
>> (setq-local fill-paragraph-function 'fill-region-as-paragraph-semlf)
>
> Neither, AFAIU?  Those two are commands, not fill-paragraph functions.
> They don't implement the protocol defined for fill-paragraph
> functions.

It should be possible to use `fill-paragraph-semlf' as
`fill-paragraph-function'.

It takes a JUSTIFY argument, which is required by:

  (funcall function justify)

in the specific case in `fill-paragraph' ("2. Try
fill-paragraph-function.").

  ;; 2. Try fill-paragraph-function.
  (and (not (eq fill-paragraph-function t))
            (or fill-paragraph-function
                (and (minibufferp (current-buffer))
                     (= 1 (point-min))))
            (let ((function (or fill-paragraph-function
                                ;; In the minibuffer, don't count
                                ;; the width of the prompt.
                                'fill-minibuffer-function))
                  ;; If fill-paragraph-function is set, it probably
                  ;; takes care of comments and stuff.  If not, it
                  ;; will have to set fill-paragraph-handle-comment
                  ;; back to t explicitly or return nil.
                  (fill-paragraph-handle-comment nil)
                  (fill-paragraph-function t))
              (funcall function justify)))

It also returns a non-nil value to avoid executing the following cases
in `fill-paragraph'.

Is there anything missing?

        Roi




This bug report was last modified 18 days ago.

Previous Next


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