From 613b698eb45f733645a5a9ba1b497c6c76c7868b Mon Sep 17 00:00:00 2001 From: =?utf8?q?Zbigniew=20J=C4=99drzejewski-Szmek?= Date: Wed, 29 May 2024 12:47:41 +0200 Subject: [PATCH] basic/fd-util: drop report from fd_cloexec_many fd_cloexec_many promised to report if work was done, but that code was not effective, because it always reported true if any fds were open. But no callers care about the return value, so let's just drop this. --- src/basic/fd-util.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/src/basic/fd-util.c b/src/basic/fd-util.c index f9cac3d99e9..c33cc6f783b 100644 --- a/src/basic/fd-util.c +++ b/src/basic/fd-util.c @@ -212,9 +212,6 @@ int fd_cloexec_many(const int fds[], size_t n_fds, bool cloexec) { continue; RET_GATHER(r, fd_cloexec(*fd, cloexec)); - - if (r >= 0) - r = 1; /* report if we did anything */ } return r; -- 2.47.3