Re: Elisp manual: Note that created faces cannot be removed.

[Alan Mackenzie]

Hello, Eli.

On Sat, Jul 20, 2019 at 21:41:20 +0300, Eli Zaretskii wrote:
> > Date: Sat, 20 Jul 2019 21:39:31 +0300
> > From: Eli Zaretskii <address@hidden>
> > Cc: address@hidden

> > > +Note that once you have defined a face with @code{defface}, you cannot
> > > +later remove this face except by restarting Emacs.

> > I think without some index entries leading to it, this sentence will
> > be lost in the ocean of the information in the manual.

Well, this page was the first one I scanned whilst seeking a remove-face
function.

> Also, defface is not the only way of defining a face, unlike what the
> proposed text seems to imply.

OK.  How about this slightly amended patch?



diff --git a/doc/lispref/display.texi b/doc/lispref/display.texi
index 276d60b21a..4ae0bba723 100644
--- a/doc/lispref/display.texi
+++ b/doc/lispref/display.texi
@@ -2476,6 +2476,10 @@ Defining Faces
 usual procedure is to define a face with @code{defface}, and then use
 its name directly.
 
+@cindex face (non-removability of)
+Note that once you have defined a face (usually with @code{defface}),
+you cannot later remove this face except by restarting Emacs.
+
 @defmac defface face spec doc [keyword value]@dots{}
 This macro declares @var{face} as a named face whose default face spec
 is given by @var{spec}.  You should not quote the symbol @var{face},


-- 
Alan Mackenzie (Nuremberg, Germany).