Subject: Re: bug#15745: [PATCH] Define semantic-idle-symbol-highlight-face with defface

From: Stefan Monnier <[email-masked]>
Date: Mon, 28 Oct 2013 21:38:41 -0400

> -(defvar semantic-idle-symbol-highlight-face 'region
> -  "Face used for highlighting local symbols.")
> +(defface semantic-idle-symbol-highlight-face
> +  '((((class color) (background dark))
> +     ;; Put this back to something closer to black later.
> +     (:background "gray20"))
> +    (((class color) (background light))
> +     (:background "gray90")))
> +  "Face used for highlighting local symbols."
> +  :group 'semantic-faces)

Just define the new face to inherit from `region', so as to preserve the
old behavior:

(defface semantic-idle-symbol-highlight
  '((t :inherit region))
  "Face used for highlighting local symbols.")

        Stefan