GNU bug report logs -
#77173
[PATCH] Instruct users to use customize or setopt for user options that have :set-er
Previous Next
Reported by: Visuwesh <visuweshm <at> gmail.com>
Date: Sat, 22 Mar 2025 12:21: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 #23 received at 77173 <at> debbugs.gnu.org (full text, mbox):
> From: Visuwesh <visuweshm <at> gmail.com>
> Cc: 77173 <at> debbugs.gnu.org
> Date: Sat, 22 Mar 2025 19:17:01 +0530
>
> [Saturday March 22, 2025] Eli Zaretskii wrote:
>
> >> From: Visuwesh <visuweshm <at> gmail.com>
> >> Date: Sat, 22 Mar 2025 17:50:31 +0530
> >>
> >> Seeing the discussion in bug#77157, and another recent discussion about
> >> the same, WDYT about the attached that automatically adds the "use
> >> setopt" filler text?
> >>
> >> This is what doc-view-epub-font-size says with the patch applied:
> >>
> >> doc-view-epub-font-size is a variable defined in ‘doc-view.el’.
> >>
> >> Its value is nil
> >>
> >> Font size in points for EPUB layout.
> >>
> >> This variable was introduced, or its default value was changed, in
> >> version 29.1 of Emacs.
> >> You can customize this variable.
> >> Setting this variable directly will not usually take effect;
> >> use either customize or ‘setopt’, or call ‘doc-view-custom-set-epub-font-size’.
> >>
> >> The word 'customize' in the last line is buttonised the same way as the
> >> one in "You can customize this variable'.
> >
> > What does this do when the setter is an anonymous function? We have
> > quite a lot of these when a defcustom needs a :set function.
>
> It currently does nothing, thanks for catching this. I forgot to
> consider lambdas when I wrote the patch earlier. I think we can leave
> out the "or call XXX" part if :set is a function?
I don't see any reasonable alternative.
This bug report was last modified 106 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.