diff options
| author | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2018-04-20 15:36:20 +0200 |
|---|---|---|
| committer | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2018-06-04 11:50:44 +0200 |
| commit | d94a24ca2ea769755beaed0659b966b1ec75c8d4 (patch) | |
| tree | e809995a39930fbc317fecc96e403c13f76737f0 /src/import/curl-util.c | |
| parent | 00bfe67f6b178056bffcfa9fbfb04ca3fa809989 (diff) | |
| download | systemd-d94a24ca2ea769755beaed0659b966b1ec75c8d4.tar.gz | |
Add macro for checking if some flags are set
This way we don't need to repeat the argument twice.
I didn't replace all instances. I think it's better to leave out:
- asserts
- comparisons like x & y == x, which are mathematically equivalent, but
here we aren't checking if flags are set, but if the argument fits in the
flags.
Diffstat (limited to 'src/import/curl-util.c')
| -rw-r--r-- | src/import/curl-util.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/import/curl-util.c b/src/import/curl-util.c index 2fba6e29dd..c19a01a266 100644 --- a/src/import/curl-util.c +++ b/src/import/curl-util.c @@ -37,7 +37,7 @@ static int curl_glue_on_io(sd_event_source *s, int fd, uint32_t revents, void *u translated_fd = PTR_TO_FD(hashmap_get(g->translate_fds, FD_TO_PTR(fd))); - if ((revents & (EPOLLIN|EPOLLOUT)) == (EPOLLIN|EPOLLOUT)) + if (FLAGS_SET(revents, EPOLLIN | EPOLLOUT)) action = CURL_POLL_INOUT; else if (revents & EPOLLIN) action = CURL_POLL_IN; |