Re: display-supports-face-attributes-p bug?

[Richard Stallman]

I installed a feature to make it possible for term/*.el to say
that the terminal does not really support underlining.
It is not elegant but it should do the job.

I also changed the underline face as follows:

(defface underline '((((supports :underline t))
                      :underline t)
                     (((supports :weight bold))
                      :weight bold)
                     (t :underline t))
  "Basic underlined face."
  :group 'basic-faces)

I think this is the right way for such faces to work.
It seems to work, but I have not installed it yet.