From 4a6a5421113ab662a665c62ed6a24b61a5a36950 Mon Sep 17 00:00:00 2001 From: Jakub Hrozek Date: Fri, 23 Sep 2011 08:47:53 +0200 Subject: Multiline macro cleanup This is mostly a cosmetic patch. The purpose of wrapping a multi-line macro in a do { } while(0) is to make the macro usable as a regular statement, not a compound statement. When the while(0) is terminated with a semicolon, the do { } while(0); block becomes a compound statement again. --- src/providers/dp_pam_data_util.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'src/providers/dp_pam_data_util.c') diff --git a/src/providers/dp_pam_data_util.c b/src/providers/dp_pam_data_util.c index 7aca559f..7ac50c4e 100644 --- a/src/providers/dp_pam_data_util.c +++ b/src/providers/dp_pam_data_util.c @@ -33,7 +33,7 @@ goto failed; \ } \ } \ -} while(0); +} while(0) #define PD_MEM_COPY(el, size) do { \ if (old_pd->el != NULL) { \ @@ -43,7 +43,7 @@ goto failed; \ } \ } \ -} while(0); +} while(0) static const char *pamcmd2str(int cmd) { switch (cmd) { -- cgit