bug#15909: 24.3.50; please fix `read-face-name' brain-dead PROMPT handling

[Drew Adams <drew.adams@oracle.com>]

In Emacs 20, the PROMPT arg was just passed to `completing-read'.  So
you used ": " at the end.  As usual in Emacs.  Simple.  Sure.  No
nonsense.

In Emacs 22, `read-face-name' decided to become "smart": It required
callers NOT to end the PROMPT with a space.  Dumb.

In Emacs 24, `read-face-name' decided to become even "smarter": It
requires callers NOT to end the PROMPT with EITHER a space or a colon.
It systematically adds ": " to PROMPT.  Dumber.

This is silly.  It means that any 3rd-party code that passes a prompt to
`read-face-name' has to fiddle with it to get the desired result for
different Emacs versions.  To what end?  What good is accomplished by
this silliness?

Please add some code similar to the following to `read-face-name', so it
DTRT.

 (when (save-match-data (string-match ": $" prompt))
   (setq prompt  (substring prompt 0 -2)))

Feel free to make this code even smarter - make it accommodate prompts
of all sorts, adding ": " or ":" or " " ONLY AS NEEDED, not
systematically.  After a decade or so of silliness, we should be able to
get this right finally.

If there were no silly history, the right approach would of course be to
have callers include ": " in the PROMPT arg, as usual in Emacs.
`read-face-name' (or some utility function it calls) could massage the
prompt to fiddle with default values or whatever.

It's not too late to make that the new interface.  But we still need to
fix the problem of callers that must work across Emacs versions.

So please update the behavior (and the doc string) to expect PROMPT to
have its own ": ", but still accommodate PROMPTs that do not have it by
adding ": ", ":", or " " as needed.  Yes, this is a mess.  And a
gratuitous one, to boot.


In GNU Emacs 24.3.50.1 (i686-pc-mingw32)
 of 2013-11-12 on LEG570
Windowing system distributor `Microsoft Corp.', version 6.1.7601
Configured using:
 `configure --enable-checking 'CFLAGS=-O0 -g3' CPPFLAGS=-DGLYPH_DEBUG=1'