bug#13686: hi-yellow vs. hi-lock-1

[Drew Adams]

> > Oh, but you said that "hi-yellow is for highlighting...in 
> > yellow".  In that case, it is not also about using other face 
> > attributes.  Time to choose.
> 
> `hi-yellow' is an example of a face that is immutable.  It is
> defconstface, so to speak.

As I said before, and at the very beginning as well, I believe: if it were in
fact constant then such a name might be apt.

Is it in fact immutable?  I didn't think so, and I see nothing in the code that
prevents changing it.  On the contrary, it seems to be a normal, customizable
face.

And that's the point.  `hi-yellow' is not a great name for a customizable face.

(And if it were truly immutable then there would be more that is constant about
it than just its yellow color.  Whether the name should then reflect only
"yellow" could be based on whether the color were the most important constant
attribute.)

If you really want an immutable face with some yellow attribute(s), then enforce
that in the code.  If you instead define a normal face  (i.e., customizable,
with no restrictions) then don't confuse users by calling it "yellow" - because
it's not.

Just one opinion, of course.