emacs: Fix two bugs in reply
authorAdam Wolfe Gordon <awg+notmuch@xvx.ca>
Sun, 1 Apr 2012 15:24:21 +0000 (09:24 -0600)
committerDavid Bremner <bremner@debian.org>
Mon, 2 Apr 2012 20:45:27 +0000 (17:45 -0300)
commit3737ca6e268e6f45353bfbcc4ac4b1d548c5908d
treeeb32c649366927334f38ad63b92ccca3a7642682
parentb45b72aa14394ba346a958da7a74303951b41ed4
emacs: Fix two bugs in reply

Bug 1: Replying from alternate addresses
----------------------------------------

The reply code was inconsistent in its use of symbols and strings for
header names being passed to message.el functions. This caused the
From header to be lookup up incorrectly, causing an additional From
header to be added with the user's primary address instead of the
correct alternate address.

This is fixed by using symbols everywhere, i.e. never using strings
for header names when interacting with message.el.

This change also removes our use of `mail-header`, since we don't use
it anywhere else, and using assq makes it clear how the header lists
are expected to work.

Bug 2: Duplicate headers in emacs 23.2
--------------------------------------

The message.el code in emacs 23.2 assumes that header names will
always be passed as symbols, so our use of strings caused
problems. The symptom was that on 23.2 (and presumably on earlier
versions) the reply message would end up with two of some headers.

Converting everything to symbols also fixes this issue.
emacs/notmuch-lib.el
emacs/notmuch-mua.el
test/emacs