bug#9029: 24.0.50; Elisp Info for defface predicate type is incorrect

bug#9029: 24.0.50; Elisp Info for defface predicate type is incorrect

[Alexis Layton]

Start info, menu Elisp, menu Faces, menu defining faces

The documentation for the predicate clause 'type' states that (type
graphic) will match all graphical displays (x, w32, presumably ns), but
in fact this is not implemented.

A look at faces.el in face-spec-set-match-display indicates that it is
intended to be changed, but is clearly not implemented as such.

The function docs for defface do not mention 'graphic' as a type value,
only the Elisp info does. (Have not checked other places.)

In my opinion it implementing graphic would be useful and the right
thing to do, so I'd be happier if someone did so rather than just
fixing the Elisp info doc, but that depends on the complexity.


In GNU Emacs 24.0.50.2 (i686-pc-linux-gnu, GTK+ Version 2.12.12)
 of 2011-06-27 on resistance-is-character-forming.permabit.com
Windowing system distributor `The X.Org Foundation', version 11.0.10402000
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: C
  value of $XMODIFIERS: nil
  locale-coding-system: nil
  default enable-multibyte-characters: t

Major mode: Fundamental

Minor modes in effect:
  tooltip-mode: t
  mouse-wheel-mode: t
  menu-bar-mode: t
  file-name-shadow-mode: t
  global-font-lock-mode: t
  font-lock-mode: t
  blink-cursor-mode: t
  auto-composition-mode: t
  auto-encryption-mode: t
  auto-compression-mode: t
  line-number-mode: t
  transient-mark-mode: t

Recent input:
<escape> x e m a c s - b u <tab> <backspace> <backspace> 
<backspace> <backspace> <backspace> <backspace> <backspace> 
<backspace> r e p o r t - e m <tab> <return>

Recent messages:
Loading /permabit/user/alex/.emacs.d/custom-23.el (source)...
Loading grep...done
Loading /permabit/user/alex/.emacs.d/custom-23.el (source)...done
Loading /permabit/user/alex/.emacs.d/alex-custom.el (source)...
Loading alex-perforce...done
Loading /permabit/user/alex/.emacs.d/alex-custom.el (source)...done
For information about GNU Emacs and the GNU system, type C-h C-a.

Load-path shadows:
~/.emacs/python-mode hides ~/.emacs.d/python-mode
~/.emacs.d/diff-mode hides /usr/local/share/emacs/24.0.50/lisp/vc/diff-mode

Features:
(shadow sort gnus-util time-date mail-extr message sendmail format-spec
rfc822 mml easymenu mml-sec mm-decode mm-bodies mm-encode mail-parse
rfc2231 rfc2047 rfc2045 ietf-drums mm-util mail-prsvr mailabbrev
mail-utils gmm-utils mailheader emacsbug p4 uniquify advice help-fns
advice-preload grep compile comint regexp-opt ring cus-start cus-load
tooltip ediff-hook vc-hooks lisp-float-type mwheel x-win x-dnd tool-bar
dnd fontset image fringe lisp-mode register page menu-bar rfn-eshadow
timer select scroll-bar mouse jit-lock font-lock syntax facemenu
font-core frame cham georgian utf-8-lang misc-lang vietnamese tibetan
thai tai-viet lao korean japanese hebrew greek romanian slovak czech
european ethiopic indian cyrillic chinese case-table epa-hook
jka-cmpr-hook help simple abbrev minibuffer loaddefs button faces
cus-face files text-properties overlay sha1 md5 base64 format env
code-pages mule custom widget hashtable-print-readable backquote
make-network-process dbusbind dynamic-setting system-font-setting
font-render-setting move-toolbar gtk x-toolkit x multi-tty emacs)

bug#9029: 24.0.50; Elisp Info for defface predicate type is incorrect

[Lars Magne Ingebrigtsen]

Alexis Layton <alex@permabit.com> writes:

> In my opinion it implementing graphic would be useful and the right
> thing to do, so I'd be happier if someone did so rather than just
> fixing the Elisp info doc, but that depends on the complexity.

I've now implemented `graphic'.

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