diff options
| author | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2021-04-14 17:44:47 +0200 |
|---|---|---|
| committer | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2021-04-14 17:45:11 +0200 |
| commit | 75029e150b47543b5c352298170847d39489c3fc (patch) | |
| tree | 8ec02193e3bf1af9a5dc4d09a054fe0914f99380 /src/basic/log.h | |
| parent | shared/module-util: fix errno value passed to log function (diff) | |
| download | systemd-75029e150b47543b5c352298170847d39489c3fc.tar.xz systemd-75029e150b47543b5c352298170847d39489c3fc.zip | |
Do not try to return 0 from log_debug()
As @yuwata correctly points out, this became broken when log_debug()
started returning -EIO. I wanted to preserve this pattern, but it turns
out it is not very widely used, and preserving it would make the whole
thing, already quite complicated, even more complex.
log_debug() is made like log_info() and friends, and returns void.
Diffstat (limited to 'src/basic/log.h')
| -rw-r--r-- | src/basic/log.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/basic/log.h b/src/basic/log.h index 9a57fcd0f1..81330166a4 100644 --- a/src/basic/log.h +++ b/src/basic/log.h @@ -221,7 +221,7 @@ void log_assert_failed_return( int log_emergency_level(void); /* Normal logging */ -#define log_debug(...) log_full_errno_zerook(LOG_DEBUG, 0, __VA_ARGS__) +#define log_debug(...) log_full(LOG_DEBUG, __VA_ARGS__) #define log_info(...) log_full(LOG_INFO, __VA_ARGS__) #define log_notice(...) log_full(LOG_NOTICE, __VA_ARGS__) #define log_warning(...) log_full(LOG_WARNING, __VA_ARGS__) |