* Re: [Emacs-diffs] master 0a6a2fb: Improve documentation of several recent changes
[not found] ` <20180616101226.5451B2043E@vcs0.savannah.gnu.org>
@ 2018-06-16 14:34 ` Daniel Colascione
2018-06-16 15:08 ` Eli Zaretskii
0 siblings, 1 reply; 4+ messages in thread
From: Daniel Colascione @ 2018-06-16 14:34 UTC (permalink / raw)
To: emacs-devel, Eli Zaretskii
On 06/16/2018 03:12 AM, Eli Zaretskii wrote:
> branch: master
> commit 0a6a2fb1d0b02a5ac442cf6b344e35696aac7eb7
> Author: Eli Zaretskii <eliz@gnu.org>
> Commit: Eli Zaretskii <eliz@gnu.org>
>
> Improve documentation of several recent changes
face-filters-always-match is deliberately not mentioned in the elisp
manual. It's for face-remap.el's use only. Please remove the additional
text about it.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Emacs-diffs] master 0a6a2fb: Improve documentation of several recent changes
2018-06-16 14:34 ` [Emacs-diffs] master 0a6a2fb: Improve documentation of several recent changes Daniel Colascione
@ 2018-06-16 15:08 ` Eli Zaretskii
2018-06-16 15:21 ` Daniel Colascione
0 siblings, 1 reply; 4+ messages in thread
From: Eli Zaretskii @ 2018-06-16 15:08 UTC (permalink / raw)
To: Daniel Colascione; +Cc: emacs-devel
> From: Daniel Colascione <dancol@dancol.org>
> Date: Sat, 16 Jun 2018 07:34:13 -0700
>
> On 06/16/2018 03:12 AM, Eli Zaretskii wrote:
> > branch: master
> > commit 0a6a2fb1d0b02a5ac442cf6b344e35696aac7eb7
> > Author: Eli Zaretskii <eliz@gnu.org>
> > Commit: Eli Zaretskii <eliz@gnu.org>
> >
> > Improve documentation of several recent changes
>
> face-filters-always-match is deliberately not mentioned in the elisp
> manual. It's for face-remap.el's use only. Please remove the additional
> text about it.
I'll take it as a THANK YOU.
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Emacs-diffs] master 0a6a2fb: Improve documentation of several recent changes
2018-06-16 15:08 ` Eli Zaretskii
@ 2018-06-16 15:21 ` Daniel Colascione
2018-06-16 15:58 ` Eli Zaretskii
0 siblings, 1 reply; 4+ messages in thread
From: Daniel Colascione @ 2018-06-16 15:21 UTC (permalink / raw)
To: Eli Zaretskii; +Cc: Daniel Colascione, emacs-devel
>> From: Daniel Colascione <dancol@dancol.org>
>> Date: Sat, 16 Jun 2018 07:34:13 -0700
>>
>> On 06/16/2018 03:12 AM, Eli Zaretskii wrote:
>> > branch: master
>> > commit 0a6a2fb1d0b02a5ac442cf6b344e35696aac7eb7
>> > Author: Eli Zaretskii <eliz@gnu.org>
>> > Commit: Eli Zaretskii <eliz@gnu.org>
>> >
>> > Improve documentation of several recent changes
>>
>> face-filters-always-match is deliberately not mentioned in the elisp
>> manual. It's for face-remap.el's use only. Please remove the additional
>> text about it.
>
> I'll take it as a THANK YOU
Well, that too. :-)
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Emacs-diffs] master 0a6a2fb: Improve documentation of several recent changes
2018-06-16 15:21 ` Daniel Colascione
@ 2018-06-16 15:58 ` Eli Zaretskii
0 siblings, 0 replies; 4+ messages in thread
From: Eli Zaretskii @ 2018-06-16 15:58 UTC (permalink / raw)
To: Daniel Colascione; +Cc: emacs-devel
> Date: Sat, 16 Jun 2018 08:21:04 -0700
> From: "Daniel Colascione" <dancol@dancol.org>
> Cc: "Daniel Colascione" <dancol@dancol.org>,
> emacs-devel@gnu.org
>
> >> face-filters-always-match is deliberately not mentioned in the elisp
> >> manual. It's for face-remap.el's use only. Please remove the additional
> >> text about it.
> >
> > I'll take it as a THANK YOU
>
> Well, that too. :-)
You are welcome.
As for documentation of face-filters-always-match: if we want it to be
an internal variable, let's name it as such, and let's say in the doc
string something like "Internal use only." Then removing it from the
manual will be a logical step.
See, that's why documenting features by those who write the code is
preferable: just by looking at the variable's name, doc string, and
uses (or lack thereof), it was hard for me to say what was the
original intent.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-06-16 15:58 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20180616101224.6099.14566@vcs0.savannah.gnu.org>
[not found] ` <20180616101226.5451B2043E@vcs0.savannah.gnu.org>
2018-06-16 14:34 ` [Emacs-diffs] master 0a6a2fb: Improve documentation of several recent changes Daniel Colascione
2018-06-16 15:08 ` Eli Zaretskii
2018-06-16 15:21 ` Daniel Colascione
2018-06-16 15:58 ` Eli Zaretskii
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).