diff options
| author | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2018-06-11 16:02:03 +0200 |
|---|---|---|
| committer | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2018-06-13 10:52:40 +0200 |
| commit | 5d904a6aaaceae7fe2f11d6b848b0dd45e3fd1c4 (patch) | |
| tree | b030bb7f74e44a28676f91d05faae95ed0c80a94 /src/busctl | |
| parent | 37e744e8661c3edd8b8efa723a3d3c71f73ef542 (diff) | |
| download | systemd-5d904a6aaaceae7fe2f11d6b848b0dd45e3fd1c4.tar.gz | |
tree-wide: drop !! casts to booleans
They are not needed, because anything that is non-zero is converted
to true.
C11:
> 6.3.1.2: When any scalar value is converted to _Bool, the result is 0 if the
> value compares equal to 0; otherwise, the result is 1.
https://stackoverflow.com/questions/31551888/casting-int-to-bool-in-c-c
Diffstat (limited to 'src/busctl')
| -rw-r--r-- | src/busctl/busctl.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/src/busctl/busctl.c b/src/busctl/busctl.c index 5c65147cac..d6844057d3 100644 --- a/src/busctl/busctl.c +++ b/src/busctl/busctl.c @@ -1972,7 +1972,7 @@ static int parse_argv(int argc, char *argv[]) { return r; } - arg_expect_reply = !!r; + arg_expect_reply = r; break; case ARG_AUTO_START: @@ -1982,7 +1982,7 @@ static int parse_argv(int argc, char *argv[]) { return r; } - arg_auto_start = !!r; + arg_auto_start = r; break; case ARG_ALLOW_INTERACTIVE_AUTHORIZATION: @@ -1992,7 +1992,7 @@ static int parse_argv(int argc, char *argv[]) { return r; } - arg_allow_interactive_authorization = !!r; + arg_allow_interactive_authorization = r; break; case ARG_TIMEOUT: @@ -2011,7 +2011,7 @@ static int parse_argv(int argc, char *argv[]) { return r; } - arg_augment_creds = !!r; + arg_augment_creds = r; break; case ARG_WATCH_BIND: @@ -2021,7 +2021,7 @@ static int parse_argv(int argc, char *argv[]) { return r; } - arg_watch_bind = !!r; + arg_watch_bind = r; break; case '?': |