[PATCH 3/5] cli/dump: define GZPRINTF macro and use it in place of gzprintf
Tomi Ollila
tomi.ollila at iki.fi
Mon Apr 13 01:52:25 PDT 2020
On Sun, Apr 12 2020, David Bremner wrote:
> This will at least catch errors, and can be replaced with more
> sophisticated error handling where appropriate.
> ---
> notmuch-client.h | 3 +++
> notmuch-dump.c | 24 ++++++++++++------------
> 2 files changed, 15 insertions(+), 12 deletions(-)
>
> diff --git a/notmuch-client.h b/notmuch-client.h
> index 467e1d84..55d4d526 100644
> --- a/notmuch-client.h
> +++ b/notmuch-client.h
> @@ -497,6 +497,9 @@ print_status_gzbytes (const char *loc,
> gzFile file,
> int bytes);
>
> +#define ASSERT_GZBYTES(file, bytes) ( (print_status_gzbytes(__location__, file, bytes)) ? exit(1) : 0 )
__location__ ?? never seen such a (preprocessor?) definition. could not
find any reference by search...
there are also some suspiciously looking like inconsistent spaces in that
macro above
rest of the series (before and after) look good
Tomi
More information about the notmuch
mailing list