Re: New custom faces for rmail.el

[Stefan Monnier]

> But AFAICS there are many places where we have both mistakes: 1) have a
> face *and* a variable, and 2) name them with the *-face suffix.

Having a variable is not necessarily an error.  In the case of
font-lock, this was done originally so as to allow font-lock faces to be
changed on a per-buffer basis (by making the variable buffer-local and
making it point to different faces).  Until Miles installs his changes
that add the ability to change faces on a per-buffer basis, this is
still useful.

As for the -face name, this is historical and should probably be
changed, tho when we changed the others we specifically avoided changing
the font-lock ones.


        Stefan