From: Jim Meyering <jim@meyering.net>
Date: Sun, 27 Aug 2006 16:20:52 +0000 (+0000)
Subject: * src/copy.c (copy_internal): Add comments.
X-Git-Tag: v6.2~64
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=7f535c46a6c797a4aa2be033e56f3bf75f60592a;p=thirdparty%2Fcoreutils.git

* src/copy.c (copy_internal): Add comments.
---

diff --git a/ChangeLog b/ChangeLog
index 9d7970d6bd..2f461cf953 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,7 @@
+2006-08-27  Jim Meyering  <jim@meyering.net>
+
+	* src/copy.c (copy_internal): Add comments.
+
 2006-08-26  Paul Eggert  <eggert@cs.ucla.edu>
 
 	* src/Makefile.am (AM_CPPFLAGS): Remove -I$(srcdir) and -I../lib,
diff --git a/src/copy.c b/src/copy.c
index 9ca2ae9df0..70886b3380 100644
--- a/src/copy.c
+++ b/src/copy.c
@@ -1033,7 +1033,8 @@ copy_internal (char const *src_name, char const *dst_name,
 	    }
 	}
       else
-	{
+	{ /* Here, we know that dst_name exists, at least to the point
+	     that it is XSTAT'able.  */
 	  bool return_now;
 	  bool unlink_src;
 	  bool ok = same_file_ok (src_name, &src_sb, dst_name, &dst_sb,
@@ -1067,6 +1068,8 @@ copy_internal (char const *src_name, char const *dst_name,
 		{
 		  if (x->move_mode && x->backup_type != no_backups)
 		    {
+		      /* Moving/copying a directory onto an existing
+			 non-directory is ok only with --backup.  */
 		    }
 		  else
 		    {
@@ -1101,6 +1104,8 @@ copy_internal (char const *src_name, char const *dst_name,
 		{
 		  if (x->move_mode && x->backup_type != no_backups)
 		    {
+		      /* Moving/copying a non-directory onto an existing
+			 directory is ok only with --backup.  */
 		    }
 		  else
 		    {