emacs: do not put quoted reply in primary selection
authorMark Walters <markwalters1009@gmail.com>
Sun, 17 Nov 2013 12:04:09 +0000 (12:04 +0000)
committerDavid Bremner <david@tethera.net>
Wed, 20 Nov 2013 00:18:17 +0000 (20:18 -0400)
commit4eb151e26ce06f8889c6b1ca43ca79f7bfdab26d
tree0d643bde32175baa6492511d4668b630afa0397c
parent2fd7ef64baf02892a7bebfb57d593afb086145ef
emacs: do not put quoted reply in primary selection

In current emacs (24.3) select-active-regions is set to t by
default. The reply insertion code sets the region to the quoted
message to make it easy to delete (kill-region or C-w). These two
things combine to put the quoted message in the primary selection.

This is not what the user wanted and is a privacy risk (accidental
pasting of the quoted message). We can avoid some of the problems
by let-binding select-active-regions to nil. This fixes if the
primary selection was previously in a non-emacs window but not if
it was in an emacs window. To avoid the problem in the latter case
we deactivate mark.

One key test (which fails under many simpler "fixes") is: open emacs
24.3 with notmuch, open 2 windows (viewing different notmuch buffers),
highlight some text in one, and then reply to a message in the
other. In many of my earlier attempts to fix this big this test fails.
emacs/notmuch-mua.el