Re: [CEDET-devel] [RFC] pulse.el

[Daimrod <da...@gm...>]

"Eric M. Ludlam" <er...@si...> writes:

> On 05/18/2013 11:54 AM, Daimrod wrote:
>> Hello,
>>
>> 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
>> `pulse-momentary-highlight-region'.
>>
>> 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
>>    too)
>> - if it's ok to use lexical-let/closure or if I should use a global
>>    variable and properties.;
>> - which functions should be available;
>
> Hi,
>
> 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 already have a copyright assignment to Emacs.

> 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.

OK

> 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.

I didn't know that the actual implementation had a mode where the pulse
could not stop until keypress. I always thought that it had to stop when
a key was pressed because it used `sit-for'.

> CEDET uses the pulse functions for pulsing a predefined overlay, a
> region, or a line.

OK

> Thanks
> Eric

-- 
Daimrod/Greg