Re: Customizing faces with `defcustom'

[Juri Linkov]

>> It's not related to the earlier problem.  I see three possibilities to
>> handle this:
>
> My patch should handle that.

Since `invisible' is a special feature of cpp.el and it has nothing to do
with faces (i.e. it uses overlays to hide text blocks), then it seems
your patch is a better way to handle this than to use a fake face.

Your patch has one problem.  It changes the default values of elements
of `cpp-edit-list'.  Currently the default value is the face `default':

Cpp Edit List: Hide Value
INS DEL List:
            String: 
            Face: (sample) default
            Face: (sample) default
            Choice: Value Menu t

But your patch changes the default to the value `invisible':

Cpp Edit List: Hide Value
INS DEL List:
            Macro: 
            True: Choice: Value Menu invisible
            False: Choice: Value Menu invisible
            Choice: Value Menu True branch writable

I think it's better to keep old defaults.

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