aboutsummaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorLasse Collin <lasse.collin@tukaani.org>2010-09-06 10:16:24 +0300
committerLasse Collin <lasse.collin@tukaani.org>2010-09-06 10:16:24 +0300
commit58f55131820d2e08a1a6beb9ec0ee2378044eb30 (patch)
tree3468310a63b0544fb28665dda93dd352158ae8ed /src
parentxz: Update the comment about NetBSD in file_io.c. (diff)
downloadxz-58f55131820d2e08a1a6beb9ec0ee2378044eb30.tar.xz
xz: Improve a comment.
Diffstat (limited to 'src')
-rw-r--r--src/xz/file_io.c7
1 files changed, 4 insertions, 3 deletions
diff --git a/src/xz/file_io.c b/src/xz/file_io.c
index 10829841..6e24c58a 100644
--- a/src/xz/file_io.c
+++ b/src/xz/file_io.c
@@ -370,15 +370,16 @@ io_open_src_real(file_pair *pair)
}
#ifdef O_NOFOLLOW
- // Give an understandable error message in if reason
+ // Give an understandable error message if the reason
// for failing was that the file was a symbolic link.
//
// Note that at least Linux, OpenBSD, Solaris, and Darwin
- // use ELOOP to indicate if O_NOFOLLOW was the reason
+ // use ELOOP to indicate that O_NOFOLLOW was the reason
// that open() failed. Because there may be
// directories in the pathname, ELOOP may occur also
// because of a symlink loop in the directory part.
- // So ELOOP doesn't tell us what actually went wrong.
+ // So ELOOP doesn't tell us what actually went wrong,
+ // and this stupidity went into POSIX-1.2008 too.
//
// FreeBSD associates EMLINK with O_NOFOLLOW and
// Tru64 uses ENOTSUP. We use these directly here