summaryrefslogtreecommitdiff
path: root/src/shared/discover-image.c
diff options
context:
space:
mode:
authorZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>2021-12-07 15:02:55 +0100
committerZbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>2021-12-15 16:19:13 +0100
commitc7f0d9e5acef3f62db3640d5ab7446241c022b35 (patch)
treed8312bdb6ba90d0277306ba8f3b08dc391759e5a /src/shared/discover-image.c
parent372e8592c6e3a891174344be0899a91769ef61af (diff)
downloadsystemd-c7f0d9e5acef3f62db3640d5ab7446241c022b35.tar.gz
tree-wide: make FOREACH_DIRENT_ALL define the iterator variable
The variable is not useful outside of the loop (it'll always be null after the loop is finished), so we can declare it inline in the loop. This saves one variable declaration and reduces the chances that somebody tries to use the variable outside of the loop. For consistency, 'de' is used everywhere for the var name.
Diffstat (limited to 'src/shared/discover-image.c')
-rw-r--r--src/shared/discover-image.c1
1 files changed, 0 insertions, 1 deletions
diff --git a/src/shared/discover-image.c b/src/shared/discover-image.c
index 9c07cda8ee..268d910214 100644
--- a/src/shared/discover-image.c
+++ b/src/shared/discover-image.c
@@ -543,7 +543,6 @@ int image_discover(
NULSTR_FOREACH(path, image_search_path[class]) {
_cleanup_free_ char *resolved = NULL;
_cleanup_closedir_ DIR *d = NULL;
- struct dirent *de;
r = chase_symlinks_and_opendir(path, root, CHASE_PREFIX_ROOT, &resolved, &d);
if (r == -ENOENT)