bug#8378: 23.2; eval-defun vs defface docstring

bug#8378: 23.2; eval-defun vs defface docstring

[Kevin Ryde]

When M-x eval-defun is used on a defface, the face docstring is not
updated.  For example in the scratch buffer M-x eval-defun on

    (defface my-test '((t (:foreground "blue")))
      "This is a blue face docstring.")

and then on

    (defface my-test '((t (:foreground "red")))
      "This is a red face docstring.")

followed by

    M-x describe-face my-test

shows

    "This is a blue face docstring"

in the help, where I expected it t be "red face" after evaluating the
second form.

The face colour and the display, is updated correctly (to red) as far as
I can tell, but not the docstring somehow.


In GNU Emacs 23.2.1 (i486-pc-linux-gnu, GTK+ Version 2.20.0)
 of 2010-05-16 on raven, modified by Debian
configured using `configure  '--build' 'i486-linux-gnu' '--build' 'i486-linux-gnu' '--prefix=/usr' '--sharedstatedir=/var/lib' '--libexecdir=/usr/lib' '--localstatedir=/var/lib' '--infodir=/usr/share/info' '--mandir=/usr/share/man' '--with-pop=yes' '--enable-locallisppath=/etc/emacs23:/etc/emacs:/usr/local/share/emacs/23.2/site-lisp:/usr/local/share/emacs/site-lisp:/usr/share/emacs/23.2/site-lisp:/usr/share/emacs/site-lisp:/usr/share/emacs/23.2/leim' '--with-x=yes' '--with-x-toolkit=gtk' '--with-toolkit-scroll-bars' 'build_alias=i486-linux-gnu' 'CFLAGS=-DDEBIAN -g -O2' 'LDFLAGS=-g' 'CPPFLAGS=''

Important settings:
  value of $LC_ALL: nil
  value of $LC_COLLATE: nil
  value of $LC_CTYPE: nil
  value of $LC_MESSAGES: nil
  value of $LC_MONETARY: nil
  value of $LC_NUMERIC: nil
  value of $LC_TIME: nil
  value of $LANG: en_AU
  value of $XMODIFIERS: nil
  locale-coding-system: iso-latin-1-unix
  default enable-multibyte-characters: t

bug#8378: 23.2; eval-defun vs defface docstring

[Lars Magne Ingebrigtsen]

Kevin Ryde <user42@zip.com.au> writes:

> When M-x eval-defun is used on a defface, the face docstring is not
> updated.  For example in the scratch buffer M-x eval-defun on
>
>     (defface my-test '((t (:foreground "blue")))
>       "This is a blue face docstring.")
>
> and then on
>
>     (defface my-test '((t (:foreground "red")))
>       "This is a red face docstring.")
>
> followed by
>
>     M-x describe-face my-test
>
> shows
>
>     "This is a blue face docstring"

I've now fixed this in Emacs 24.

-- 
(domestic pets only, the antidote for overdose, milk.)
  bloggy blog http://lars.ingebrigtsen.no/