>     Why is it setting deactivate-mark?  I thought that was only set by the
>     sort of interactive command that shouldn't be used in hooks etc.,
>     which makes this problem sound more like a simple bug rather than a
>     design problem....

> Any change to the buffer sets deactivate-mark.  So flyspell's
> post-command-hook should bind deactivate-mark to prevent that.

I think this is too drastic.  We should first and foremost avoid running
"ispell -V" every time the post-command-hook is run.  And maybe we should
let bind deactivate-mark around ispell-check-version.  But maybe not around
the whole flyspell-post-command-hook.


        Stefan


> Does this fix it?

> *** flyspell.el       01 Nov 2005 20:34:12 -0500      1.83
> --- flyspell.el       03 Nov 2005 11:46:05 -0500      
> ***************
> *** 895,901 ****
>   (defun flyspell-post-command-hook ()
>     "The `post-command-hook' used by flyspell to check a word in-the-fly."
>     (interactive)
> !   (let ((command this-command))
>       (if (flyspell-check-pre-word-p)
>       (save-excursion
>         '(flyspell-debug-signal-pre-word-checked)
> --- 895,903 ----
>   (defun flyspell-post-command-hook ()
>     "The `post-command-hook' used by flyspell to check a word in-the-fly."
>     (interactive)
> !   (let ((command this-command)
> !     ;; Prevent anything we do from affecting the mark.
> !     deactivate-mark)
>       (if (flyspell-check-pre-word-p)
>       (save-excursion
>         '(flyspell-debug-signal-pre-word-checked)


_______________________________________________
Emacs-pretest-bug mailing list
Emacs-pretest-bug@gnu.org
http://lists.gnu.org/mailman/listinfo/emacs-pretest-bug

Reply via email to