>
> Given that users may get their config wrong, do you think we should really test
> the value of vm-summary-highlight-face to ensure it is a legit face name i.e.
>
> (if (and selected (facep vm-summary-highlight-face))
Yes, it is a good idea.
Picking up on another issue you asked, faces shouldn't have names ending in
"-face", as prescribed in the elisp manual. Face variables normally do.
> highlight- face to ensure it is a legit face name i.e. highlight- face))
> Given that users may get their config wrong, do you think we should really test
> the value of vm-summary-
>
> (if (and selected (facep vm-summary-
Yes, it is a good idea.
Picking up on another issue you asked, faces shouldn't have names ending in
"-face", as prescribed in the elisp manual. Face variables normally do.