On Fri 2016-07-15 02:23:20 +0200, David Bremner wrote: > - accumulate an error string. With talloc_asprintf_append, this is > not _too_ terrible. Making a second logging function [1] that > didn't clear the log buffer but appended would maybe make sense > (aside from contradicting what I said in the previous message). > This would still need some status return to alert the caller. [...] > [1]: untested: > > void > _notmuch_database_log_append (notmuch_database_t *notmuch, > const char *format, > ...) > { > va_list va_args; > > va_start (va_args, format); > > if (notmuch->status_string) > notmuch->status_string = talloc_vasprintf_append (notmuch->status_string, format, va_args) > else > notmuch->status_string = talloc_vasprintf (notmuch, format, va_args); > > va_end (va_args); > } If Someoneā¢ were to publish a changeset with this feature, i'd certainly make use of it in this series. thanks for the suggestion, David. --dkg