diff options
| author | Jakub Hrozek <jhrozek@redhat.com> | 2011-09-23 08:47:53 +0200 |
|---|---|---|
| committer | Stephen Gallagher <sgallagh@redhat.com> | 2011-09-28 09:06:57 -0400 |
| commit | 4a6a5421113ab662a665c62ed6a24b61a5a36950 (patch) | |
| tree | 037524123a5e1685d5e2942d1d5983896146a06a /src/providers/dp_pam_data_util.c | |
| parent | db2c71bd61b7e1610ea1a178ef05059ef952685a (diff) | |
| download | sssd_unused-4a6a5421113ab662a665c62ed6a24b61a5a36950.tar.gz sssd_unused-4a6a5421113ab662a665c62ed6a24b61a5a36950.tar.xz sssd_unused-4a6a5421113ab662a665c62ed6a24b61a5a36950.zip | |
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.
Diffstat (limited to 'src/providers/dp_pam_data_util.c')
| -rw-r--r-- | src/providers/dp_pam_data_util.c | 4 |
1 files changed, 2 insertions, 2 deletions
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) { |