diff options
author | Lasse Collin <lasse.collin@tukaani.org> | 2010-09-06 10:16:24 +0300 |
---|---|---|
committer | Lasse Collin <lasse.collin@tukaani.org> | 2010-09-06 10:16:24 +0300 |
commit | 58f55131820d2e08a1a6beb9ec0ee2378044eb30 (patch) | |
tree | 3468310a63b0544fb28665dda93dd352158ae8ed /src | |
parent | xz: Update the comment about NetBSD in file_io.c. (diff) | |
download | xz-58f55131820d2e08a1a6beb9ec0ee2378044eb30.tar.xz |
xz: Improve a comment.
Diffstat (limited to 'src')
-rw-r--r-- | src/xz/file_io.c | 7 |
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 |