From 837c9427de10ddcb5295f3840ead2a6c3aa61e69 Mon Sep 17 00:00:00 2001 From: Xavier Del Campo Romero Date: Thu, 8 Oct 2020 08:47:15 +0200 Subject: [PATCH] Add missing do/while (0) on convenience macros --- include/dynstr.h | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/include/dynstr.h b/include/dynstr.h index b2e102c..79edc58 100644 --- a/include/dynstr.h +++ b/include/dynstr.h @@ -163,10 +163,11 @@ void dynstr_free(struct dynstr *d); * Convenience macro that calls dynstr_append and returns its error code if failed. */ #define dynstr_append_or_ret(...) \ + do \ { \ const enum dynstr_err err = dynstr_append(__VA_ARGS__); \ if (err != DYNSTR_OK) return err; \ - } + } while (0) /** * Convenience macro that calls dynstr_append and returns zero if failed. @@ -196,10 +197,11 @@ void dynstr_free(struct dynstr *d); * Convenience macro that calls dynstr_prepend and returns its error code if failed. */ #define dynstr_prepend_or_ret(...) \ + do \ { \ const enum dynstr_err err = dynstr_prepend(__VA_ARGS__); \ if (err != DYNSTR_OK) return err; \ - } + } while (0) /** * Convenience macro that calls dynstr_prepend and returns zero if failed.