Re: delphi-other-face mismatch

[Juri Linkov]

>     `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?

There are other customizable face-valued variables (e.g. `locate-header-face'
in locate.el) which have mismatched nil default values.  Adding a construct
like `(choice (const :tag "None" nil) face)' only to variables with nil
default values doesn't help, because users might want to set to nil those
face variables whose default value is not nil.  How users can do this?
Setting the value to the face `default' (which produces the desirable
visual effect) is not exactly the same as a nil value.

Maybe customize should interpret an empty face input field as a nil
value?  (Currently it signals an error "Invalid face: " in this case.)

-- 
Juri Linkov
http://www.jurta.org/emacs/