* bug#36670: 27.0.50; Wishlist: describe-face should use the new help-buffer infrastructure
@ 2019-07-15 16:14 Lars Ingebrigtsen
2019-09-20 22:48 ` Lars Ingebrigtsen
0 siblings, 1 reply; 8+ messages in thread
From: Lars Ingebrigtsen @ 2019-07-15 16:14 UTC (permalink / raw)
To: 36670
M-x describe-variable (and -function) uses a new infrastructure based on
help-fns-describe-{variable,function}-functions that registers a bunch
of little bits to be output, which gives more flexibility.
describe-face should use the same thing, and be moved to help-fns.el.
In GNU Emacs 27.0.50 (build 2, x86_64-pc-linux-gnu, GTK+ Version 3.22.30)
of 2019-07-15 built on sandy
Repository revision: d0e558c0025b295bbd8c51122600d1083ed6d391
Repository branch: master
Windowing system distributor 'The X.Org Foundation', version 11.0.11906000
System Description: Ubuntu 18.04.2 LTS
--
(domestic pets only, the antidote for overdose, milk.)
bloggy blog: http://lars.ingebrigtsen.no
^ permalink raw reply [flat|nested] 8+ messages in thread* bug#36670: 27.0.50; Wishlist: describe-face should use the new help-buffer infrastructure
2019-07-15 16:14 bug#36670: 27.0.50; Wishlist: describe-face should use the new help-buffer infrastructure Lars Ingebrigtsen
@ 2019-09-20 22:48 ` Lars Ingebrigtsen
2019-09-21 6:41 ` Eli Zaretskii
0 siblings, 1 reply; 8+ messages in thread
From: Lars Ingebrigtsen @ 2019-09-20 22:48 UTC (permalink / raw)
To: 36670
Lars Ingebrigtsen <larsi@gnus.org> writes:
> M-x describe-variable (and -function) uses a new infrastructure based on
> help-fns-describe-{variable,function}-functions that registers a bunch
> of little bits to be output, which gives more flexibility.
>
> describe-face should use the same thing, and be moved to help-fns.el.
I have now done this on the trunk, as previously requested by Stefan M.
--
(domestic pets only, the antidote for overdose, milk.)
bloggy blog: http://lars.ingebrigtsen.no
^ permalink raw reply [flat|nested] 8+ messages in thread* bug#36670: 27.0.50; Wishlist: describe-face should use the new help-buffer infrastructure
2019-09-20 22:48 ` Lars Ingebrigtsen
@ 2019-09-21 6:41 ` Eli Zaretskii
2019-09-21 7:44 ` Lars Ingebrigtsen
0 siblings, 1 reply; 8+ messages in thread
From: Eli Zaretskii @ 2019-09-21 6:41 UTC (permalink / raw)
To: Lars Ingebrigtsen; +Cc: 36670
> From: Lars Ingebrigtsen <larsi@gnus.org>
> Date: Sat, 21 Sep 2019 00:48:09 +0200
>
> Lars Ingebrigtsen <larsi@gnus.org> writes:
>
> > M-x describe-variable (and -function) uses a new infrastructure based on
> > help-fns-describe-{variable,function}-functions that registers a bunch
> > of little bits to be output, which gives more flexibility.
> >
> > describe-face should use the same thing, and be moved to help-fns.el.
>
> I have now done this on the trunk, as previously requested by Stefan M.
What does this mean in practice? I see nothing about this in NEWS or
in the manuals, which makes me wonder how is this "more flexible". I
guess there were some discussions, but neither the log message nor the
bug discussion mentions any other sources of information. Could you
please point to those discussions, or fill the void by explaining what
additional flexibility do we gain? This is for future forensics, if
for nothing else.
Thanks.
^ permalink raw reply [flat|nested] 8+ messages in thread* bug#36670: 27.0.50; Wishlist: describe-face should use the new help-buffer infrastructure
2019-09-21 6:41 ` Eli Zaretskii
@ 2019-09-21 7:44 ` Lars Ingebrigtsen
2019-09-21 8:22 ` Eli Zaretskii
0 siblings, 1 reply; 8+ messages in thread
From: Lars Ingebrigtsen @ 2019-09-21 7:44 UTC (permalink / raw)
To: Eli Zaretskii; +Cc: 36670
Eli Zaretskii <eliz@gnu.org> writes:
> What does this mean in practice? I see nothing about this in NEWS or
> in the manuals, which makes me wonder how is this "more flexible". I
> guess there were some discussions, but neither the log message nor the
> bug discussion mentions any other sources of information. Could you
> please point to those discussions, or fill the void by explaining what
> additional flexibility do we gain? This is for future forensics, if
> for nothing else.
I don't think this was discussed much at all -- it relates to the
rewrite Stefan did in April, and he mentioned (in a comment to a fix to
`describe-face') that that function should be rewritten in a similar way.
commit 896e5802160c2797e689a7565599ebb1bd171295
Author: Stefan Monnier <monnier@iro.umontreal.ca>
Date: Fri Apr 12 12:37:00 2019 -0400
* lisp/help-fns.el (help-fns-describe-variable-functions): New hook
(help-fns--compiler-macro, help-fns--parent-mode, help-fns--obsolete)
(help-fns--interactive-only): Indent output by 2 spaces.
(help-fns--side-effects): New function extracted from
describe-function-1.
(help-fns-describe-function-functions): Use it.
(help-fns--first-release, help-fns--mention-first-release): New functions.
(help-fns-function-description-header): Keymaps and macros can't
be interactive.
(help-fns--ensure-empty-line): New function.
(describe-function-1): Use it.
(help-fns--var-safe-local, help-fns--var-risky)
(help-fns--var-ignored-local, help-fns--var-file-local)
(help-fns--var-watchpoints, help-fns--var-obsolete)
(help-fns--var-alias, help-fns--var-bufferlocal): New functions,
extacted from describe-variable.
(describe-variable): Run help-fns-describe-variable-functions instead.
--
(domestic pets only, the antidote for overdose, milk.)
bloggy blog: http://lars.ingebrigtsen.no
^ permalink raw reply [flat|nested] 8+ messages in thread* bug#36670: 27.0.50; Wishlist: describe-face should use the new help-buffer infrastructure
2019-09-21 7:44 ` Lars Ingebrigtsen
@ 2019-09-21 8:22 ` Eli Zaretskii
2019-09-21 8:38 ` Lars Ingebrigtsen
0 siblings, 1 reply; 8+ messages in thread
From: Eli Zaretskii @ 2019-09-21 8:22 UTC (permalink / raw)
To: Lars Ingebrigtsen, Stefan Monnier; +Cc: 36670
> From: Lars Ingebrigtsen <larsi@gnus.org>
> Cc: 36670@debbugs.gnu.org
> Date: Sat, 21 Sep 2019 09:44:51 +0200
>
> Eli Zaretskii <eliz@gnu.org> writes:
>
> > What does this mean in practice? I see nothing about this in NEWS or
> > in the manuals, which makes me wonder how is this "more flexible". I
> > guess there were some discussions, but neither the log message nor the
> > bug discussion mentions any other sources of information. Could you
> > please point to those discussions, or fill the void by explaining what
> > additional flexibility do we gain? This is for future forensics, if
> > for nothing else.
>
> I don't think this was discussed much at all -- it relates to the
> rewrite Stefan did in April, and he mentioned (in a comment to a fix to
> `describe-face') that that function should be rewritten in a similar way.
>
> commit 896e5802160c2797e689a7565599ebb1bd171295
> Author: Stefan Monnier <monnier@iro.umontreal.ca>
> Date: Fri Apr 12 12:37:00 2019 -0400
>
> * lisp/help-fns.el (help-fns-describe-variable-functions): New hook
>
> (help-fns--compiler-macro, help-fns--parent-mode, help-fns--obsolete)
> (help-fns--interactive-only): Indent output by 2 spaces.
> (help-fns--side-effects): New function extracted from
> describe-function-1.
> (help-fns-describe-function-functions): Use it.
> (help-fns--first-release, help-fns--mention-first-release): New functions.
> (help-fns-function-description-header): Keymaps and macros can't
> be interactive.
> (help-fns--ensure-empty-line): New function.
> (describe-function-1): Use it.
> (help-fns--var-safe-local, help-fns--var-risky)
> (help-fns--var-ignored-local, help-fns--var-file-local)
> (help-fns--var-watchpoints, help-fns--var-obsolete)
> (help-fns--var-alias, help-fns--var-bufferlocal): New functions,
> extacted from describe-variable.
> (describe-variable): Run help-fns-describe-variable-functions instead.
So we are making changes without a good understanding what kind of
flexibility it gains us?
^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#36670: 27.0.50; Wishlist: describe-face should use the new help-buffer infrastructure
2019-09-21 8:22 ` Eli Zaretskii
@ 2019-09-21 8:38 ` Lars Ingebrigtsen
2019-09-21 9:05 ` Eli Zaretskii
0 siblings, 1 reply; 8+ messages in thread
From: Lars Ingebrigtsen @ 2019-09-21 8:38 UTC (permalink / raw)
To: Eli Zaretskii; +Cc: Stefan Monnier, 36670
Eli Zaretskii <eliz@gnu.org> writes:
> So we are making changes without a good understanding what kind of
> flexibility it gains us?
Looking at the change, the flexibility seems kinda obvious -- the
humongous help functions are split into many smaller ones so that they
can be rearranged more easily?
--
(domestic pets only, the antidote for overdose, milk.)
bloggy blog: http://lars.ingebrigtsen.no
^ permalink raw reply [flat|nested] 8+ messages in thread
* bug#36670: 27.0.50; Wishlist: describe-face should use the new help-buffer infrastructure
2019-09-21 8:38 ` Lars Ingebrigtsen
@ 2019-09-21 9:05 ` Eli Zaretskii
2019-09-21 9:10 ` Lars Ingebrigtsen
0 siblings, 1 reply; 8+ messages in thread
From: Eli Zaretskii @ 2019-09-21 9:05 UTC (permalink / raw)
To: Lars Ingebrigtsen; +Cc: monnier, 36670
> From: Lars Ingebrigtsen <larsi@gnus.org>
> Cc: Stefan Monnier <monnier@iro.umontreal.ca>, 36670@debbugs.gnu.org
> Date: Sat, 21 Sep 2019 10:38:32 +0200
>
> Eli Zaretskii <eliz@gnu.org> writes:
>
> > So we are making changes without a good understanding what kind of
> > flexibility it gains us?
>
> Looking at the change, the flexibility seems kinda obvious -- the
> humongous help functions are split into many smaller ones so that they
> can be rearranged more easily?
So this is just a cleanup of the internals?
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2019-09-21 9:10 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-07-15 16:14 bug#36670: 27.0.50; Wishlist: describe-face should use the new help-buffer infrastructure Lars Ingebrigtsen
2019-09-20 22:48 ` Lars Ingebrigtsen
2019-09-21 6:41 ` Eli Zaretskii
2019-09-21 7:44 ` Lars Ingebrigtsen
2019-09-21 8:22 ` Eli Zaretskii
2019-09-21 8:38 ` Lars Ingebrigtsen
2019-09-21 9:05 ` Eli Zaretskii
2019-09-21 9:10 ` Lars Ingebrigtsen
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).