Re: [PATCH 1/2] CLI: add print_status_database

Subject: Re: [PATCH 1/2] CLI: add print_status_database

Date: Wed, 30 Mar 2016 22:34:18 -0300

To: Tomi Ollila, notmuch@notmuchmail.org

Cc:

From: David Bremner

Tomi Ollila <tomi.ollila@iki.fi> writes:

>
> LGTM (whole series).

Pushed the series
>
> I was going to complain about the naming of 'print_status_database()' 
> but David already got through (in 39c54df2) print_status_query() so
> that ship sailed already (perhaps that it happened is good :).
>

Hmm, well, it's not API, so if we want to rename them we can.
Of course, the longer we leave it, the more pain.

d

Thread: