GNU bug report logs - #46193
Enabling Auto Fill mode

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 46193 in the body.
You can then email your comments to 46193 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

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

From: jai-bholeki <at> gmx.com
To: bug-gnu-emacs <at> gnu.org
Subject: Enabling Auto Fill mode
Date: Sat, 30 Jan 2021 15:41:03 +0100

In the documentation for (auto-fill-mode &optional ARG), it says that
if called from Lisp, enable the mode if ARG is omitted or nil.

Enabling a mode using nil seems counter-intuitive to me.

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

From: Philipp Stephani <p.stephani2 <at> gmail.com>
To: jai-bholeki <at> gmx.com
Cc: 46193 <at> debbugs.gnu.org
Subject: Re: bug#46193: Enabling Auto Fill mode
Date: Sat, 30 Jan 2021 19:31:43 +0100

Am Sa., 30. Jan. 2021 um 17:14 Uhr schrieb <jai-bholeki <at> gmx.com>:
>
>
> In the documentation for (auto-fill-mode &optional ARG), it says that
> if called from Lisp, enable the mode if ARG is omitted or nil.
>
> Enabling a mode using nil seems counter-intuitive to me.
>


Maybe, but it's the universal convention for minor modes. See "Minor
mode conventions" in the ELisp manual.

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

From: jai-bholeki <at> gmx.com
To: Philipp Stephani <p.stephani2 <at> gmail.com>
Cc: 46193 <at> debbugs.gnu.org
Subject: Re: bug#46193: Enabling Auto Fill mode
Date: Sat, 30 Jan 2021 20:55:20 +0100

I have noticed that enabling a minor-mode using texinfo-mode-hook fails after
setting auto-fill comments using a hook.

(add-hook 'texinfo-mode-hook
	  (lambda () ((set (make-local-variable 'comment-auto-fill-only-comments) t)))
(auto-fill-mode)

;; crucible-mode not enabled after setting comment-auto-fill-only-comments
(add-hook 'texinfo-mode-hook 'crucible-mode)




> Sent: Sunday, January 31, 2021 at 6:31 AM
> From: "Philipp Stephani" <p.stephani2 <at> gmail.com>
> To: jai-bholeki <at> gmx.com
> Cc: 46193 <at> debbugs.gnu.org
> Subject: bug#46193: Enabling Auto Fill mode
>
> Am Sa., 30. Jan. 2021 um 17:14 Uhr schrieb <jai-bholeki <at> gmx.com>:
> >
> >
> > In the documentation for (auto-fill-mode &optional ARG), it says that
> > if called from Lisp, enable the mode if ARG is omitted or nil.
> >
> > Enabling a mode using nil seems counter-intuitive to me.
> >
>
>
> Maybe, but it's the universal convention for minor modes. See "Minor
> mode conventions" in the ELisp manual.
>
>
>
>

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

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Philipp Stephani <p.stephani2 <at> gmail.com>
Cc: 46193 <at> debbugs.gnu.org, jai-bholeki <at> gmx.com
Subject: Re: bug#46193: Enabling Auto Fill mode
Date: Mon, 01 Feb 2021 10:19:22 +0100

Philipp Stephani <p.stephani2 <at> gmail.com> writes:

> Am Sa., 30. Jan. 2021 um 17:14 Uhr schrieb <jai-bholeki <at> gmx.com>:
>>
>> In the documentation for (auto-fill-mode &optional ARG), it says that
>> if called from Lisp, enable the mode if ARG is omitted or nil.
>>
>> Enabling a mode using nil seems counter-intuitive to me.
>
> Maybe, but it's the universal convention for minor modes. See "Minor
> mode conventions" in the ELisp manual.

Indeed, so I'm closing this bug report.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no

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

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: jai-bholeki <at> gmx.com
Cc: 46193 <at> debbugs.gnu.org, Philipp Stephani <p.stephani2 <at> gmail.com>
Subject: Re: bug#46193: Enabling Auto Fill mode
Date: Mon, 01 Feb 2021 10:20:15 +0100

jai-bholeki <at> gmx.com writes:

> I have noticed that enabling a minor-mode using texinfo-mode-hook fails after
> setting auto-fill comments using a hook.
>
> (add-hook 'texinfo-mode-hook
> 	  (lambda () ((set (make-local-variable
> 'comment-auto-fill-only-comments) t)))
> (auto-fill-mode)
>
> ;; crucible-mode not enabled after setting comment-auto-fill-only-comments
> (add-hook 'texinfo-mode-hook 'crucible-mode)

(add-hook 'texinfo-mode-hook
	  (lambda () ((set (make-local-variable 'comment-auto-fill-only-comments) t)))

That's not valid syntax -- you have one extra pair of () around the
`set' function call.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no

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

From: jai-bholeki <at> gmx.com
To: Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: 46193 <at> debbugs.gnu.org, Philipp Stephani <p.stephani2 <at> gmail.com>
Subject: Re: bug#46193: Enabling Auto Fill mode
Date: Mon, 1 Feb 2021 12:59:25 +0100

I have now made the following function.  Would I better make the
functionality to be buffer-local?

(defvar dfv-break-long-lines-state nil)

(defun cycle-break-long-lines-din ()
  "Breaks long lines using auto-fill tool."
  (interactive)
  ;;
  (pcase dfv-break-long-lines-state
    ;;
    (1 (setq fill-column 72)
       (setq comment-auto-fill-only-comments t)
       (auto-fill-mode)
       (message "%s" "Enable: Break only comments")
       (setq dfv-break-long-lines-state 2))
    ;;
    (2 (setq comment-auto-fill-only-comments nil)
       (message "%s" "Enable: Break all long lines")
       (setq dfv-break-long-lines-state 1))
    ;;
    (_ (setq comment-auto-fill-only-comments nil)
       (auto-fill-mode 0)
       (message "%s" "Disable: Break long lines")
       (setq dfv-break-long-lines-state 0)) ))

(defun break-long-lines-din ()
  "Breaks long lines using auto-fill."
  (setq dfv-break-long-lines-state 1)
  (cycle-break-long-lines-din))



> Sent: Monday, February 01, 2021 at 9:20 PM
> From: "Lars Ingebrigtsen" <larsi <at> gnus.org>
> To: jai-bholeki <at> gmx.com
> Cc: "Philipp Stephani" <p.stephani2 <at> gmail.com>, 46193 <at> debbugs.gnu.org
> Subject: Re: bug#46193: Enabling Auto Fill mode
>
> jai-bholeki <at> gmx.com writes:
>
> > I have noticed that enabling a minor-mode using texinfo-mode-hook fails after
> > setting auto-fill comments using a hook.
> >
> > (add-hook 'texinfo-mode-hook
> > 	  (lambda () ((set (make-local-variable
> > 'comment-auto-fill-only-comments) t)))
> > (auto-fill-mode)
> >
> > ;; crucible-mode not enabled after setting comment-auto-fill-only-comments
> > (add-hook 'texinfo-mode-hook 'crucible-mode)
>
> (add-hook 'texinfo-mode-hook
> 	  (lambda () ((set (make-local-variable 'comment-auto-fill-only-comments) t)))
>
> That's not valid syntax -- you have one extra pair of () around the
> `set' function call.
>
> --
> (domestic pets only, the antidote for overdose, milk.)
>    bloggy blog: http://lars.ingebrigtsen.no
>

Previous Next

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