To test this, do M-x debug-on-entry read-face-name, and then M-x
customize-face font-lock-comment-face. Then, put point on the
second "sample" text (next to the foreground field), and do M-:
(read-face-name "foo"). Step through the debugger, and you'll see
that the ((foreground-color . "Firebrick")) case falls
through the crack.
I believe that it is seeming to do the right thing, for the wrong reason.
If the code reliably does the right thing, it isn't wrong. Perhaps it
could be clearer, or perhaps it could use a comment to explain this
case.
_______________________________________________
emacs-pretest-bug mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/emacs-pretest-bug
