[PATCH 1/2] Make `notmuch-show-clean-address' parsing-error-proof.
    Carl Worth 
    cworth at cworth.org
       
    Fri Jun  3 14:10:54 PDT 2011
    
    
  
On Thu, 12 May 2011 17:56:25 +0400, Dmitry Kurochkin <dmitry.kurochkin at gmail.com> wrote:
> Mail-header-parse-address may fail for an invalid address.
> Before the change, this would result in empty notmuch-show buffer
> with an error message like: Scan error: "Unbalanced parentheses".
> The patch wraps the function in condition-case and returns
> unchanged address in case of error.
Thanks for the patch, Dmitry. I've just pushed this.
And thanks as well for the test case. I actually pushed the test-case
commit before the bug-fix commit. I like to do things in that order so
that I can:
	1. Apply the new test case
	2. Run "make test" and see the failure
	3. Apply the bug fix
	4. Run "make test" and see the test pass
With a sequence like that, it's very easy for me to feel very
comfortable committing the code.
-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/20110603/89821ae3/attachment-0001.pgp>
    
    
More information about the notmuch
mailing list