Tomi Ollila <tomi.ollila at iki.fi> writes:
>> + ret = notmuch_database_open_verbose (path,
>> + NOTMUCH_DATABASE_MODE_READ_WRITE,
>> + ¬much,
>> + &message);
>> if (ret) {
>> + if (status_cb) status_cb (message, closure);
>> goto DONE;
>> }
>
> Is this ever printing the message (if any?)
Printing messages is the whole point of status_cb. It does switch the
output from the current stderr to stdout, which is weird. On the other
hand it avoids changing the api by adding
notmuch_database_compact_verbose.
And of course if we care this should free message after calling
status_cb, like the other places.
d