On Wed, 31 Oct 2012, David Bremner <david@tethera.net> wrote: > Jani Nikula <jani@nikula.org> writes: > >> Hi Damien, please send this patch as part of the series adding >> notmuch-labeler. Out of context, without adding any users of the >> function, it's a bit difficult to evaluate. > > For what it's worth (and I'm definitely not claiming any kind final word > here) I did tell Damien that he was welcome to send pre-requisite > patches in a seperate series. Of course, if you feel like you can't > really review what's there (or it's just too much work), that's your > call. I don't disagree with sending pre-requisite patches first. And I am definitely not asking to polish all of the follow up work at once. I should have made myself more clear about that. But I do think it would be helpful to see what the added code will be used for. We may be able to provide better and more productive review of the big picture, instead of focusing on the details of the small snippet here. BR, Jani.