xz: Improve a comment.
authorLasse Collin <lasse.collin@tukaani.org>
Mon, 6 Sep 2010 07:16:24 +0000 (10:16 +0300)
committerLasse Collin <lasse.collin@tukaani.org>
Mon, 6 Sep 2010 07:16:24 +0000 (10:16 +0300)
src/xz/file_io.c

index 1082984..6e24c58 100644 (file)
@@ -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