[PATCH] Save and restore point explicitly in `notmuch-wash-toggle-invisible-action'.

Carl Worth cworth at cworth.org
Tue May 24 15:57:32 PDT 2011


On Tue, 24 May 2011 18:43:41 -0400, Austin Clements <amdragon at mit.edu> wrote:
> Saving point this way is a bit dangerous, though.  For example, if
> you're near the end of the buffer and shorten the label, attempting to
> restore the point could result in an error (or, a more benign example:
> the cursor could wind up outside the label so pressing RET repeatedly
> won't toggle it).

Without the patch to change save-excursion to an integer, point is
already moving outside the button, (so that repeatedly pressing RET
doesn't toggle).

I'm exploring a proper fix now to get reliable behavior.

-Carl
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://notmuchmail.org/pipermail/notmuch/attachments/20110524/0b19e500/attachment.pgp>


More information about the notmuch mailing list