diff options
| author | David Goldblatt <davidgoldblatt@fb.com> | 2017-07-21 13:34:45 -0700 |
|---|---|---|
| committer | David Goldblatt <davidtgoldblatt@gmail.com> | 2017-07-22 09:38:19 -0700 |
| commit | a9f7732d45c22ca7d22bed6ff2eaeb702356884e (patch) | |
| tree | 0472fe6cc282a7fd82b0debe36a1ced5fb2d2e89 /test | |
| parent | aa6c2821374f6dd6ed2e628c06bc08b0c4bc485c (diff) | |
| download | platform_external_jemalloc_new-a9f7732d45c22ca7d22bed6ff2eaeb702356884e.tar.gz platform_external_jemalloc_new-a9f7732d45c22ca7d22bed6ff2eaeb702356884e.tar.bz2 platform_external_jemalloc_new-a9f7732d45c22ca7d22bed6ff2eaeb702356884e.zip | |
Logging: allow logging with empty varargs.
Currently, the log macro requires at least one argument after the format string,
because of the way the preprocessor handles varargs macros. We can hide some of
that irritation by pushing the extra arguments into a varargs function.
Diffstat (limited to 'test')
| -rw-r--r-- | test/unit/log.c | 16 |
1 files changed, 14 insertions, 2 deletions
diff --git a/test/unit/log.c b/test/unit/log.c index 6db256f1..053fea41 100644 --- a/test/unit/log.c +++ b/test/unit/log.c @@ -170,13 +170,25 @@ TEST_BEGIN(test_logs_if_no_init) { } TEST_END +/* + * This really just checks to make sure that this usage compiles; we don't have + * any test code to run. + */ +TEST_BEGIN(test_log_only_format_string) { + if (false) { + static log_var_t l = LOG_VAR_INIT("str"); + log(l, "No arguments follow this format string."); + } +} +TEST_END + int main(void) { - return test( test_log_disabled, test_log_enabled_direct, test_log_enabled_indirect, test_log_enabled_global, - test_logs_if_no_init); + test_logs_if_no_init, + test_log_only_format_string); } |
