[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Proposal to change cursor appearance to indicate region acti
From: |
David Kastrup |
Subject: |
Re: [PATCH] Proposal to change cursor appearance to indicate region activation |
Date: |
Fri, 30 Jan 2015 10:19:50 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.0.50 (gnu/linux) |
Kelly Dean <address@hidden> writes:
> David Kastrup wrote:
>> Multiple independent use cases. add-hook/remove-hook is a mechanism for
>> organizing independent use cases for one feature, but there is no such
>> mechanism for organizing independent use cases for the varhook feature
>> in your implementation even though you actually use add-hook. But it
>> requires first individually allocating, naming, and using a hook for any
>> variable you might want to varhook into.
>
> IIUC, you mean independent uses of varhook might choose different
> symbols for the hook. It seems that would be solved by the convention
> of using the symbol ⌜foo-varhook⌝ as the hook for ⌜foo⌝; all
> independent uses would add/remove their functions on that same hook.
A convention is not an interface.
> I guess another solution would be to put the car of the list of
> functions directly in a dedicated varhook slot for the target symbol,
> rather than indirecting through a regular hook. That would increase
> the size of each symbol from 24 to 28 bytes (on 32-bit platforms). But
> even in my main Emacs session, which has been up for 54 days (with
> 300MB reserved memory), I only have 10k symbols, so an extra 40kB of
> memory usage isn't much overhead.
I have 40k right now, and this session has been up for few minutes
(admittedly, using desktop-load). But at any rate, this does not appear
like a feature that should be a central part of the data/operation in a
production Emacs, and consequently it should not be a required part of
the implementation of a production Emacs feature either.
I also have to warn you that there is little point in trying to convince
or coax me since I more or less have the authority of a court jester.
So it is pointless to argue away problems with me since it does not help
getting the code accepted.
--
David Kastrup
- [PATCH] Run hook when variable is set, Kelly Dean, 2015/01/28
- Re: [PATCH] Proposal to change cursor appearance to indicate region activation, Kelly Dean, 2015/01/28
- Re: [PATCH] Proposal to change cursor appearance to indicate region activation, David Kastrup, 2015/01/28
- Re: [PATCH] Proposal to change cursor appearance to indicate region activation, David Kastrup, 2015/01/28
- Re: [PATCH] Proposal to change cursor appearance to indicate region activation, Kelly Dean, 2015/01/29
- Re: [PATCH] Proposal to change cursor appearance to indicate region activation, David Kastrup, 2015/01/29
- Re: [PATCH] Proposal to change cursor appearance to indicate region activation, Kelly Dean, 2015/01/30
- Re: [PATCH] Proposal to change cursor appearance to indicate region activation,
David Kastrup <=
- Re: [PATCH] Proposal to change cursor appearance to indicate region activation, Kelly Dean, 2015/01/30
- Re: [PATCH] Proposal to change cursor appearance to indicate region activation, David Kastrup, 2015/01/30
- Re: [PATCH] Proposal to change cursor appearance to indicate region activation, Kelly Dean, 2015/01/30
Re: [PATCH] Run hook when variable is set, Stefan Monnier, 2015/01/28
- Re: [PATCH] Run hook when variable is set, Kelly Dean, 2015/01/29
- Re: [PATCH] Run hook when variable is set, Lars Ingebrigtsen, 2015/01/29
- Re: [PATCH] Run hook when variable is set, Stefan Monnier, 2015/01/29
- Re: [PATCH] Run hook when variable is set, Kelly Dean, 2015/01/30
- Re: [PATCH] Run hook when variable is set, Stefan Monnier, 2015/01/30
- Re: [PATCH] Run hook when variable is set, Kelly Dean, 2015/01/31