diff options
author | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2022-10-07 16:19:21 +0200 |
---|---|---|
committer | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2022-10-11 16:59:00 +0200 |
commit | b4096cecffd13491848b84bc27f00dbee06af52e (patch) | |
tree | f77ff3d40d693e8bd396848179ca09a5af9184be /man/sd_bus_new.xml | |
parent | 29c45dc4348e7db61aa80ba1657cbc2d8b1a19ee (diff) | |
download | systemd-b4096cecffd13491848b84bc27f00dbee06af52e.tar.gz |
man: recommend %m over strerror()
The need to set errno is very very ugly, but at least it is thread-safe and
works correctly. Using strerror() is likely to be wrong, so let's not recommend
that. People who do a lot of logging would provide use some wrapper that sets
errno like we do, so nudge people towards %m.
I tested that all the separate .c files compile cleanly.
Diffstat (limited to 'man/sd_bus_new.xml')
-rw-r--r-- | man/sd_bus_new.xml | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/man/sd_bus_new.xml b/man/sd_bus_new.xml index 41964640b6..b45106168c 100644 --- a/man/sd_bus_new.xml +++ b/man/sd_bus_new.xml @@ -119,8 +119,10 @@ int r; … r = sd_bus_default(&bus); - if (r < 0) - fprintf(stderr, "Failed to allocate bus: %s\n", strerror(-r)); + if (r < 0) { + errno = -r; + fprintf(stderr, "Failed to allocate bus: %m\n"); + } … }</programlisting> |