Re: read-face-name bugs

[Richard Stallman]

    2. I think the code is wrong for the case where there is a face
       property like this on the character: ((foreground-color
       "Firebrick")).

It seems to give the right results: it finds no face names
and `faces' is empty.

In what way do you think this is incorrect?

       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.