From: Lasse Collin <lasse.collin@tukaani.org>
Date: Mon, 6 Sep 2010 07:16:24 +0000 (+0300)
Subject: xz: Improve a comment.
X-Git-Tag: v5.0.0~59
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=58f55131820d2e08a1a6beb9ec0ee2378044eb30;p=xz

xz: Improve a comment.
---

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