Re: [address@hidden: Re: [Preview-latex-devel] CVS-1.42; scale function]

[Stefan Monnier]

>>>>> "David" == David Kastrup <David.Kastrup@t-online.de> writes:
> The mouse-2 feature already gets provided by a 'keymap property
> (actually, the 'local-map property since flyspell.el is lacking the
> knowledge that Emacs-21 already has the 'keymap property) on the
> overlay.  The minor map key binding is there from a time where the
> overlay keymaps did not work, and so the entire buffer got overlayed
> with a mouse-2 binding that tries to detect whether the mouse is
> pressed in an overlay and if not, tries to do what the mousepress
> would have done instead.  This fake key action ignores overlay keymaps
> and thus destroys my bindings.

Ah, I think you're right.  It probably ends up not taking
the overlay bindings into account.  I'll take a look.
Thank you for that precise description.

As for "why the minor mode map".  It's actually a little bit more subtle
than just historical: the intent is to be able to type M-TAB even
a few chars after the mistyped word in which case the overlay's
`local-map' and `keymap' are of no help.

> flyspell is trying to simulate not being active at all, but fails.  It
> is easier to let flyspell not attempt something which is absolutely
> and totally unnecessary than to try to improve its attempt at
> simulating Emacs.

I agree that the code is risky.  See above for the reason why it's done
that way.  As for whether the risk is worth the benefit, I don't know.

Richard> On the contrary, the `local-map' property is NEWER than
Richard> the `keymap' property.

Your memory is failing you, Richard.  The `keymap' property is new
in Emacs-21.


        Stefan