emacs-pretest-bug
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: delphi-other-face mismatch


From: Luc Teirlinck
Subject: Re: delphi-other-face mismatch
Date: Sun, 28 Aug 2005 14:07:02 -0500 (CDT)

Richard Stallman wrote:

       `delphi.el' has nil mismatch in:

       (defcustom delphi-other-face nil
          "*Face used to color everything else."
          :type 'face
          :group 'delphi)

   The simplest fix would be to specify a non-nil default.
   Would someone who has a feel for choosing faces
   please do that?

Why not do what similar options, like `display-time-mail-face' do in
similar situations?  I can install the trivial patch below if desired.

===File ~/delphi.el-diff====================================
*** delphi.el   01 Aug 2005 07:12:36 -0500      3.18
--- delphi.el   28 Aug 2005 13:45:45 -0500      
***************
*** 176,182 ****
  
  (defcustom delphi-other-face nil
    "*Face used to color everything else."
!   :type 'face
    :group 'delphi)
  
  (defconst delphi-directives
--- 176,182 ----
  
  (defcustom delphi-other-face nil
    "*Face used to color everything else."
!   :type '(choice (const :tag "None" nil) face)
    :group 'delphi)
  
  (defconst delphi-directives
============================================================




reply via email to

[Prev in Thread] Current Thread [Next in Thread]