GNU bug report logs - #41145
27.0.91; small issues with `display-fill-column-indicator' Customization group

Previous Next

Package: emacs;

Reported by: Philipp Stephani <p.stephani2 <at> gmail.com>

Date: Sat, 9 May 2020 08:31:01 UTC

Severity: normal

Found in version 27.0.91

Done: Stefan Monnier <monnier <at> iro.umontreal.ca>

Bug is archived. No further changes may be made.

Full log

View this message in rfc822 format

From: Stefan Monnier <monnier <at> iro.umontreal.ca>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 41145 <at> debbugs.gnu.org, p.stephani2 <at> gmail.com, Mauro Aranda <maurooaranda <at> gmail.com>
Subject: bug#41145: 27.0.91; small issues with `display-fill-column-indicator' Customization group
Date: Sat, 29 Aug 2020 11:36:12 -0400

>> I did some debugging and found that the difference is that there are no
>> custom options or faces defined in display-fill-column-indicator.el:
>> fill-column-indicator is in faces.el and
>> display-fill-column-indicator, display-fill-column-indicator-character
>> and display-fill-column-indicator-column are in xdisp.c

Which begs the question: why define the group there?

>> Or if eventually a defface or a defcustom makes its way into
>> display-fill-column-indicator.el, of course.

Actually, there is a defcustom there (in the expansion of
`define-globalized-minor-mode`).

>> Note that adding the above form will result in adding cus-edit to the
>> custom-loads property of the groups defined in cus-edit (like editing,
>> convenience, etc.).  I don't know if that is harmless.
>
> Stefan, any suggestions regarding which solution route is preferable?

I'd get rid of the file and move its content to one of the
preloaded files (simple.el?  faces.el?  fill.el? ...)


        Stefan
This bug report was last modified 4 years and 248 days ago.

Previous Next

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