summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorJim Meyering <meyering@redhat.com>2011-01-22 12:55:58 +0100
committerJim Meyering <meyering@redhat.com>2011-01-30 20:44:11 +0100
commitce3c73dddddf04a5f1e03f3830198ecc55e56c75 (patch)
tree1aa882b8682ae4e59238334ca2d21b34cb82e67f /src
parent0497a86f7001526f7cf35e0f876063a06a748e4b (diff)
downloadcoreutils-ce3c73dddddf04a5f1e03f3830198ecc55e56c75.tar.xz
copy: tweak variable name; improve a comment
* src/copy.c (copy_reg): Rename a variable to make more sense from caller's perspective: s/require_normal_copy/normal_copy_required/. This is an output-only variable, and the original name could make it look like an input (or i&o) variable.
Diffstat (limited to 'src')
-rw-r--r--src/copy.c12
1 files changed, 6 insertions, 6 deletions
diff --git a/src/copy.c b/src/copy.c
index be7fdba7b..fae8dbe0e 100644
--- a/src/copy.c
+++ b/src/copy.c
@@ -868,17 +868,17 @@ copy_reg (char const *src_name, char const *dst_name,
#endif
}
- bool require_normal_copy;
- /* Perform efficient extent copy for sparse file, fall back to the
+ bool normal_copy_required;
+ /* Perform an efficient extent-based copy, falling back to the
standard copy only if the initial extent scan fails. If the
- '--sparse=never' option was specified, we writing all data but
- use extent copy if available to efficiently read. */
+ '--sparse=never' option is specified, write all data but use
+ any extents to read more efficiently. */
if (extent_copy (source_desc, dest_desc, buf_size,
src_open_sb.st_size, make_holes,
- src_name, dst_name, &require_normal_copy))
+ src_name, dst_name, &normal_copy_required))
goto preserve_metadata;
- if (! require_normal_copy)
+ if (! normal_copy_required)
{
return_val = false;
goto close_src_and_dst_desc;