[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Run hook when variable is set
|
From: |
Kelly Dean |
|
Subject: |
Re: [PATCH] Run hook when variable is set |
|
Date: |
Thu, 29 Jan 2015 08:20:27 +0000 |
Stefan Monnier wrote:
> NACK from me.
Why? Just because of the slowdown for symbols that have property lists? Or some
additional reason?
I.e. if I eliminate the slowdown, then will you still have an objection to the
feature?
> All that just to avoid using the minor mode?
The updated patch I sent you does make dynamic-cursor-mode be a minor mode
rather than a defcustom, because you said you want it to be a minor mode. I
didn't implement varhook in order to avoid that.
> I already posted the code
> that I think should be used for that minor mode.
And I explained why your code doesn't provide a complete solution. You didn't
respond.
I implemented varhook to completely solve the problem of dynamic-cursor-mode
interfering with other uses of cursor-type in all cases, while ensuring that
dynamic-cursor-mode itself can reliably be (re)enabled in all cases.
Obviously, there are additional uses for this feature too. For just one
example, without varhook, what solution would you propose for bug #19068?
- Re: [PATCH] Proposal to change cursor appearance to indicate region activation, (continued)
- 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, 2015/01/30
- 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 <=
- 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
- Re: [PATCH] Run hook when variable is set, Stefan Monnier, 2015/01/31
Re: [PATCH] Run hook when variable is set, Richard Stallman, 2015/01/30
Re: [PATCH] Run hook when variable is set, Kelly Dean, 2015/01/31
Re: [PATCH] Run hook when variable is set, Richard Stallman, 2015/01/31
Re: [PATCH] Run hook when variable is set, Alexis, 2015/01/31