diff options
| author | Yu Watanabe <watanabe.yu+github@gmail.com> | 2023-04-28 14:05:29 +0900 |
|---|---|---|
| committer | Yu Watanabe <watanabe.yu+github@gmail.com> | 2023-04-28 14:06:00 +0900 |
| commit | 71e84b4be68746dd61b52d274da864477350fe9b (patch) | |
| tree | e269c8400db3331c99b703c6ef6e4d4ffc740bdf /src/import | |
| parent | c63dde809951496a5e9c5a568aadfb9cad95e5e5 (diff) | |
| download | systemd-71e84b4be68746dd61b52d274da864477350fe9b.tar.gz | |
copy: rename reflink_full() -> reflink_range()
The commit b640e274a7c363a2b6394c9dce5671d9404d2e2a introduced reflink()
and reflink_full(). We usually name function xyz_full() for fully
parameterized version of xyz(), and xyz() is typically a inline alias of
xyz_full(). But in this case, reflink() and reflink_full() call
different ioctl().
Moreover, reflink_full() does partial reflink, while reflink() does full
file reflink. That's super confusing.
Let's rename reflink_full() to reflink_range(), the new name is
consistent with ioctl name, and should be fine.
Diffstat (limited to 'src/import')
| -rw-r--r-- | src/import/qcow2-util.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/import/qcow2-util.c b/src/import/qcow2-util.c index 9addb5c555..c70656bc0e 100644 --- a/src/import/qcow2-util.c +++ b/src/import/qcow2-util.c @@ -69,7 +69,7 @@ static int copy_cluster( ssize_t l; int r; - r = reflink_full(sfd, soffset, dfd, doffset, cluster_size); + r = reflink_range(sfd, soffset, dfd, doffset, cluster_size); if (r >= 0) return r; |