[PATCH] emacs: Function to query the list of labels from a thread-id

Jani Nikula jani at nikula.org
Wed Oct 31 13:48:14 PDT 2012


On Wed, 31 Oct 2012, David Bremner <david at tethera.net> wrote:
> Jani Nikula <jani at 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.


More information about the notmuch mailing list