Subject: 24.3.50; read-face-name: M-o o hi-yellow fails

From: Jambunathan K <[email-masked]>
Date: Mon, 15 Apr 2013 22:44:11 +0530

M-o o hi-yellow fails with

,----
| Debugger entered--Lisp error: (wrong-type-argument stringp (hi-yellow))
|   intern((hi-yellow))
|   facemenu-add-new-face((hi-yellow))
|   facemenu-set-face((hi-yellow) nil nil)
|   call-interactively(facemenu-set-face nil nil)
|   command-execute(facemenu-set-face)
`----

The following fix works.

,----  Towards the end of `read-face-name'
| 
|     ;; Return either a list of faces or just one face.
|     (if multiple
|         (nreverse faces)
|       (car (last faces)))   <===== FIXME: `car' is necessary
`----

In GNU Emacs 24.3.50.2 (i686-pc-linux-gnu, GTK+ Version 2.20.1)
 of 2013-04-15 on debian-6.05
Bzr revision: 112293 [email-masked]
Windowing system distributor `The X.Org Foundation', version 11.0.10707000
Important settings:
  value of $LANG: en_IN
  locale-coding-system: iso-latin-1-unix
  default enable-multibyte-characters: t