summaryrefslogtreecommitdiff
path: root/io
diff options
context:
space:
mode:
authorRoland McGrath <roland@hack.frob.com>2014-10-20 15:32:26 -0700
committerRoland McGrath <roland@hack.frob.com>2014-10-20 15:32:45 -0700
commit6ab1d1ea969df0698ee413a247948fc035e12276 (patch)
tree0b997d7da555aa87e2643fb09b12051a2edaa98c /io
parente436eb790f577218f9cea15e545500e952df80ae (diff)
Tiny refactoring in fts to eliminate a warning.
Diffstat (limited to 'io')
-rw-r--r--io/fts.c16
1 files changed, 11 insertions, 5 deletions
diff --git a/io/fts.c b/io/fts.c
index c9c054d6d3..46cbb72571 100644
--- a/io/fts.c
+++ b/io/fts.c
@@ -561,6 +561,16 @@ fts_children(sp, instr)
return (sp->fts_child);
}
+static inline int
+dirent_not_directory(const struct dirent *dp)
+{
+#if defined DT_DIR && defined _DIRENT_HAVE_D_TYPE
+ return dp->d_type != DT_DIR && dp->d_type != DT_UNKNOWN;
+#else
+ return 0;
+#endif
+}
+
/*
* This is the tricky part -- do not casually change *anything* in here. The
* idea is to build the linked list of entries that are used by fts_children
@@ -759,11 +769,7 @@ mem1: saved_errno = errno;
p->fts_info = FTS_NSOK;
p->fts_accpath = cur->fts_accpath;
} else if (nlinks == 0
-#if defined DT_DIR && defined _DIRENT_HAVE_D_TYPE
- || (nostat &&
- dp->d_type != DT_DIR && dp->d_type != DT_UNKNOWN)
-#endif
- ) {
+ || (nostat && dirent_not_directory(dp))) {
p->fts_accpath =
ISSET(FTS_NOCHDIR) ? p->fts_path : p->fts_name;
p->fts_info = FTS_NSOK;