[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: face-remapping-alist client code
From: |
Stefan Monnier |
Subject: |
Re: face-remapping-alist client code |
Date: |
Mon, 02 Jun 2008 14:54:29 -0400 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/23.0.60 (gnu/linux) |
>> - the variables `text-scale-mode-lighter',
>> `text-scale-mode-remapping', `text-scale-mode-amount' and
>> `variable-pitch-mode-remapping' would make more sense as
>> automatically buffer local, as they are always buffer-local'ized when
>> used.
> I somehow got the impression that make-variable-buffer-local was frowned
> upon. Dunno if it's true.
It's perfectly fine for internal variables. It's a definite no-no when
you call it on some other package's variable. And as for variables that
are manipulated by other packages, it may be better to force them to
always use make-local-variable, otherwise they may be surprised if they
`setq' before your package was loaded.
>> - the lighter for variable-pitch-mode is a bit of redundant, isn't
>> it? Either the user has a variable-pitch face different from the
>> default one (and so he doesn't need the reminder in the modeline), or
>> the mode is not going to be useful anyway...
> I think the lighters are actually quite useful. It's nice to have an
> explicit reminders of active modes as well as implicit (and sometimes
> subtle) ones.
Either I wouldn't see it (fine), or it'd push away some other lighter
(my mode-line is usually full already).
Stefan
- Re: face-remapping-alist client code, (continued)
- Re: face-remapping-alist client code, Miles Bader, 2008/06/02
- RE: face-remapping-alist client code, Drew Adams, 2008/06/02
- Re: face-remapping-alist client code, Miles Bader, 2008/06/02
- Re: face-remapping-alist client code, Miles Bader, 2008/06/02
- Re: face-remapping-alist client code, Juanma Barranquero, 2008/06/02
- Re: face-remapping-alist client code, Miles Bader, 2008/06/02
- Re: face-remapping-alist client code, David Kastrup, 2008/06/02
- Re: face-remapping-alist client code, Stefan Monnier, 2008/06/02
- RE: face-remapping-alist client code, Drew Adams, 2008/06/02
- Re: face-remapping-alist client code, Juanma Barranquero, 2008/06/02
- Re: face-remapping-alist client code,
Stefan Monnier <=
- RE: face-remapping-alist client code, Drew Adams, 2008/06/02
Re: face-remapping-alist client code, Chong Yidong, 2008/06/02