On 05/18/2013 11:54 AM, Daimrod wrote:
> I've recently switched to a less powerful laptop and I have been annoyed
> by the pulse functionality because it made Emacs frames blur. So I have
> rewrote my own version of pulse.el which uses a timer instead of
> `sit-for'. Thanks to this, it doesn't make Emacs frames shakes and it is
> possible to move the cursor during the effect.
> ATM I've only implemented something similar to
> I would like to know:
> - if you would be interested in this new implementation of pulse.el;
> (I'm not the only one impacted, an #emacs user reported this problem
> - if it's ok to use lexical-let/closure or if I should use a global
> variable and properties.;
> - which functions should be available;
Your version of pulse sounds like a good idea. Do you have papers
allowing you to contribute software to Emacs? If so, or if you can get
papers, I'd be happy to take improvements on the pulse utility.
I am not picky about the use of lexical-let or not. Pulse should stand
alone as a utility that happens to be used by CEDET. It should support
Emacs 23 for CEDET, or you could contribute it straight into Emacs
proper where CEDET w/ Emacs/BZR could use it.
Pulse also has a mode where the region in question just highlights a
line until keypress. This is for cases where a user might think it is
too slow, or for much older versions of Emacs. A quick search shows
several users trying to use the non-pulsing version by setting
pulse-flag to nil. Perhaps your version will fix things so this doesn't
matter? It would certainly shrink the overall size of pulse.el to drop
support for that.
CEDET uses the pulse functions for pulsing a predefined overlay, a
region, or a line.