Re: read-face-name bugs

[Richard Stallman]

               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.