"Brennan Vincent" <bren...@umanwizard.com> writes: >> Since notmuch is already a talloc context, you should address the need >> for a new context somewhere (either a comment or the commit message). > I don't remember why I did this. I've never used talloc before and > probably just got confused. I'll switch to using notmuch.
I should have been more explicit. There is a good reason, namely the lifetime of those temporary strings is much shorter when using the "local" talloc context, but I had to stare at it a bit to figure that out. _______________________________________________ notmuch mailing list -- notmuch@notmuchmail.org To unsubscribe send an email to notmuch-le...@notmuchmail.org