improve from-header guessing

Dirk Hohndel hohndel at infradead.org
Sat Apr 24 12:09:56 PDT 2010


On Fri, 23 Apr 2010 11:47:04 -0700, Carl Worth <cworth at cworth.org> wrote:
> On Fri, 16 Apr 2010 13:51:40 -0700, Dirk Hohndel <hohndel at infradead.org> wrote:
> > The following two patches should address most of the concerns raised 
> > to my previous series. 
> 
> Allow me to raise new concerns then. ;-)

Any time
 
> > The first patch simply adds an interface to obtain a concatenation of
> > all instances of a specific header from an email.
> 
> I was hoping to see the "special-case value of NULL" go away with this
> change.
> 
> And I like that there's a new function to get the concatenated header,
> (I would prefer an unabbreviated name of get_concatenated_header than
> get_header_concat), but I don't like seeing all the existing callers of
> get_header updated to pass an extra 0. Instead, I'd prefer to see those
> calls unchanged, and a tiny new get_header that passes the 0 and then
> make the actual implementing function be static and named something like
> notmuch_message_file_get_header_internal.

Turns out that the way I did this was broken anyway. So we can simply
forget these patches and your concerns. I'm sure you'll raise new
concerns on the new ("rearchitected") patches.

> Both patches have some trailing whitespace. I see these easily wince I
> have the following in my ~/.gitconfig:
> 
> 	[core]
> 		whitespace = trailing-space,space-before-tab

I know. I'm trying to be better about checking whitespace pollution
before submitting things.

> Finally, I'd like to see some tests for this feature. (But we do have
> the feature already without tests, so I won't strictly block on that).

Hu? You even commited these already. Or am I reading email out of order
again? 

/D


More information about the notmuch mailing list