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/tests | |
| parent | db2c71bd61b7e1610ea1a178ef05059ef952685a (diff) | |
| download | sssd-4a6a5421113ab662a665c62ed6a24b61a5a36950.tar.gz sssd-4a6a5421113ab662a665c62ed6a24b61a5a36950.tar.bz2 sssd-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/tests')
| -rw-r--r-- | src/tests/krb5_utils-tests.c | 2 | ||||
| -rw-r--r-- | src/tests/strtonum-tests.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/src/tests/krb5_utils-tests.c b/src/tests/krb5_utils-tests.c index 4392abd7..6993398a 100644 --- a/src/tests/krb5_utils-tests.c +++ b/src/tests/krb5_utils-tests.c @@ -52,7 +52,7 @@ struct krb5child_req *kr; ret = rmdir(__dir__); \ fail_unless(ret == EOK, "rmdir [%s] failed, [%d][%s].", __dir__, \ errno, strerror(errno)); \ -} while(0); +} while(0) void setup_create_dir(void) { diff --git a/src/tests/strtonum-tests.c b/src/tests/strtonum-tests.c index 441b262c..e7d70364 100644 --- a/src/tests/strtonum-tests.c +++ b/src/tests/strtonum-tests.c @@ -58,7 +58,7 @@ #define CHECK_ZERO_ENDPTR(endptr) \ do { \ fail_unless(endptr && *endptr == '\0', "Invalid endptr"); \ - } while(0); + } while(0) /****************** * strtoint tests * |