https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=262038
--- Comment #11 from Mark Johnston <ma...@freebsd.org> --- (In reply to Mahmoud Abumandour from comment #6) Sorry for the delayed reply. This mostly looks ok to me, but I have a few more nits: - Rather than setting errno = 0 in fts_safe_readdir(), I think it's better to set readdir_errno only when readdir(2) fails. - FreeBSD style (see the style(9) man page) is to put declarations at the beginning of the function, so fts_safe_readdir() should declare "ret" at the beginning. - The comment above the readdir_errno check does not seem to be needed. -- You are receiving this mail because: You are the assignee for the bug.