GNU bug report logs - #27238
24.5; Rename `face-spec-set-2' and give it a better doc string

Previous Next

Package: emacs;

Reported by: Drew Adams <drew.adams <at> oracle.com>

Date: Sun, 4 Jun 2017 17:43:01 UTC

Severity: wishlist

Found in version 24.5

Done: Eli Zaretskii <eliz <at> gnu.org>

Bug is archived. No further changes may be made.

Full log


Message #11 received at 27238 <at> debbugs.gnu.org (full text, mbox):

From: Drew Adams <drew.adams <at> oracle.com>
To: Eli Zaretskii <eliz <at> gnu.org>, Drew Adams <drew.adams <at> oracle.com>
Cc: 27238 <at> debbugs.gnu.org
Subject: RE: bug#27238: 24.5;	Rename `face-spec-set-2' and give it a better
 doc string
Date: Sun, 4 Jun 2017 12:38:10 -0700 (PDT)
> > This is a useful, general function, not just a helper.
> 
> FWIW, I don't think it's general enough, because it is tightly couples
> with face-x-resources.

That's only the _implementation_.

(That reasoning would apply also to `face-spec-recalc', `face-spec-set',
`face-set-after-frame-default', `custom-face-set', `custom-face-save',
`custom-face-mark-to-save', `custom-face-reset-saved',
`custom-face-mark-to-reset-standard', `custom-declare-face',
`custom-theme-set-faces', `custom-reset-faces', `woman-default-faces',
...)

Unless you essentially repeat its body, it is the only way
to do what it does.  Here's one use case:

(face-spec-set-2 TARGET-FACE
                 FRAME
                 (face-spec-choose
                   (get SOURCE-FACE 'face-defface-spec)
                   FRAME))




This bug report was last modified 7 years and 348 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.