From: "Drew Adams" <drew.adams@oracle.com>
To: "'Miles Bader'" <miles@gnu.org>
Cc: 'Glenn Morris' <rgm@gnu.org>, 'Dan Nicolaescu' <dann@ics.uci.edu>,
emacs-devel@gnu.org,
'Roland Winkler' <Roland.Winkler@physik.uni-erlangen.de>
Subject: RE: faces and face variables
Date: Sun, 14 Sep 2008 12:04:51 -0700 [thread overview]
Message-ID: <005101c9169c$c5166be0$0200a8c0@us.oracle.com> (raw)
In-Reply-To: <87zlmamzj7.fsf@catnip.gol.com>
> > But I don't agree that face variables are or should be
> > "deprecated" or that all uses of face variables are
> > necessarily "fossils".
>
> 99% of "face variables" in existing code are not used the way you
> described, but instead are pointless frippery.
I never suggested otherwise. In fact, I said the same thing.
And I said clearly that all such fossils should be removed
from the existing code.
Again:
IOW, let's not confuse the effort to remove such fossils
from the Emacs code with the idea that no one should ever
use face variables. Use them in the (rare) cases where
they are useful; don't use them otherwise.
> As richard said, there can be cases where a variable
> referring to a face is called for -- but there are almost
> _always_ considered exceptions to any given rule; that does not
> make the rule wrong for the general case.
Again, that's just what I said. Please reread. No one has
claimed that the rule is "wrong for the general case".
And Richard didn't say that, BTW - I did. He merely
acknowledged that there might be such cases, and that the
particular use case I pointed out is reasonable.
The point is that we should not just provide the rule, but
also explain it. And to do that in the Elisp manual, not just
in some source code comments somewhere.
next prev parent reply other threads:[~2008-09-14 19:04 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <200809090636.m896acaT011007@sallyv1.ics.uci.edu>
[not found] ` <18630.36126.116571.102340@tfkp07.physik.uni-erlangen.de>
[not found] ` <200809091834.m89IYJrt004178@sallyv1.ics.uci.edu>
[not found] ` <18630.52279.410707.428217@tfkp07.physik.uni-erlangen.de>
[not found] ` <200809100554.m8A5sbLU020022@sallyv1.ics.uci.edu>
[not found] ` <18634.41341.360898.898779@tfkp07.physik.uni-erlangen.de>
[not found] ` <200809121750.m8CHoXar025729@sallyv1.ics.uci.edu>
[not found] ` <18634.45534.791696.614584@tfkp07.physik.uni-erlangen.de>
[not found] ` <200809130953.m8D9rEZC011379@sallyv1.ics.uci.edu>
2008-09-13 16:03 ` faces and face variables Roland Winkler
2008-09-13 18:09 ` Glenn Morris
2008-09-13 19:27 ` Roland Winkler
2008-09-13 22:43 ` Drew Adams
2008-09-14 17:02 ` Miles Bader
2008-09-14 19:04 ` Drew Adams [this message]
[not found] ` <18775.34703.848628.238267@tfkp04.physik.uni-erlangen.de>
2008-12-29 6:33 ` Proced's display/handling of process trees: request for feedback Roland Winkler
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='005101c9169c$c5166be0$0200a8c0@us.oracle.com' \
--to=drew.adams@oracle.com \
--cc=Roland.Winkler@physik.uni-erlangen.de \
--cc=dann@ics.uci.edu \
--cc=emacs-devel@gnu.org \
--cc=miles@gnu.org \
--cc=rgm@gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this external index
https://git.savannah.gnu.org/cgit/emacs.git
https://git.savannah.gnu.org/cgit/emacs/org-mode.git
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.