(Defining Faces): Recommend against face variables.

diff --git a/doc/lispref/ChangeLog b/doc/lispref/ChangeLog
index 44da8528114363d0bdb10f67c06f47f63dbc77ad..971ac59889155d0131661dc32a370b52beb5ab91 100644 (file)
--- a/doc/lispref/ChangeLog
+++ b/doc/lispref/ChangeLog
@@ -1,3 +1,7 @@
+2008-09-20  Glenn Morris  <rgm@gnu.org>
+
+       * display.texi (Defining Faces): Recommend against face variables.
+
 2008-09-16  Juanma Barranquero  <lekktu@gmail.com>
 
        * display.texi (Echo Area Customization): Fix typo.

diff --git a/doc/lispref/display.texi b/doc/lispref/display.texi
index 4e154f7f2635d208eb0c60e2f41cf437d23aa5e1..01e869234fbe41427ca280127ee0bb9dfec5a1ab 100644 (file)
--- a/doc/lispref/display.texi
+++ b/doc/lispref/display.texi
@@ -1803,6 +1803,11 @@ kind of customization item (@pxref{Customization}) which the user can
 customize using the Customization buffer (@pxref{Easy Customization,,,
 emacs, The GNU Emacs Manual}).
 
+  People are sometimes tempted to create variables whose values specify
+which faces to use (for example, Font-Lock does this).  In the vast
+majority of cases, this is not necessary, and simply using faces
+directly is preferable.
+
 @defmac defface face spec doc [keyword value]@dots{}
 This declares @var{face} as a customizable face that defaults
 according to @var{spec}.  You should not quote the symbol @var{face},