aboutgitcodebugslistschat
diff options
context:
space:
mode:
authorDavid Gibson <david@gibson.dropbear.id.au>2024-03-01 15:06:44 +1100
committerStefano Brivio <sbrivio@redhat.com>2024-03-07 23:09:15 +0100
commit137ce017896b2b64f8c45a8ddefbaed2cb71c445 (patch)
tree9f0814ae064b8ab6dae15f6af2b4b5f2d81d5721
parentbb11d1549544a9a7505f7fe699ad3949ada1d1d4 (diff)
downloadpasst-137ce017896b2b64f8c45a8ddefbaed2cb71c445.tar
passt-137ce017896b2b64f8c45a8ddefbaed2cb71c445.tar.gz
passt-137ce017896b2b64f8c45a8ddefbaed2cb71c445.tar.bz2
passt-137ce017896b2b64f8c45a8ddefbaed2cb71c445.tar.lz
passt-137ce017896b2b64f8c45a8ddefbaed2cb71c445.tar.xz
passt-137ce017896b2b64f8c45a8ddefbaed2cb71c445.tar.zst
passt-137ce017896b2b64f8c45a8ddefbaed2cb71c445.zip
iov: Improve documentation of iov_skip_bytes()
As pointed out in review, the documentation comments for iov_skip_bytes() are more confusing than they should be. Reword them, including updating parameter names, to make it clearer. Signed-off-by: David Gibson <david@gibson.dropbear.id.au> Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
-rw-r--r--iov.c28
-rw-r--r--iov.h2
2 files changed, 15 insertions, 15 deletions
diff --git a/iov.c b/iov.c
index e7b4eb7..52a7c01 100644
--- a/iov.c
+++ b/iov.c
@@ -26,31 +26,31 @@
#include "iov.h"
-/* iov_skip_bytes() - Skip the first n bytes into an IO vector
+/* iov_skip_bytes() - Skip leading bytes of an IO vector
* @iov: IO vector
* @n: Number of entries in @iov
- * @vec_offset: Total byte offset into the IO vector
- * @buf_offset: Offset into a single buffer of the IO vector
+ * @skip: Number of leading bytes of @iov to skip
+ * @offset: Offset of first unskipped byte in its @iov entry
*
- * Return: index I of individual struct iovec which contains the byte at
- * @vec_offset bytes into the vector (as though all its buffers were
- * contiguous). If @buf_offset is non-NULL, update it to the offset of
- * that byte within @iov[I] (guaranteed to be less than @iov[I].iov_len)
- * If the whole vector has <= @vec_offset bytes, return @n.
+ * Return: index I of individual struct iovec which contains the byte at @skip
+ * bytes into the vector (as though all its buffers were contiguous).
+ * If @offset is non-NULL, update it to the offset of that byte within
+ * @iov[I] (guaranteed to be less than @iov[I].iov_len) If the whole
+ * vector has <= @skip bytes, return @n.
*/
size_t iov_skip_bytes(const struct iovec *iov, size_t n,
- size_t vec_offset, size_t *buf_offset)
+ size_t skip, size_t *offset)
{
- size_t offset = vec_offset, i;
+ size_t off = skip, i;
for (i = 0; i < n; i++) {
- if (offset < iov[i].iov_len)
+ if (off < iov[i].iov_len)
break;
- offset -= iov[i].iov_len;
+ off -= iov[i].iov_len;
}
- if (buf_offset)
- *buf_offset = offset;
+ if (offset)
+ *offset = off;
return i;
}
diff --git a/iov.h b/iov.h
index e1becde..6058af7 100644
--- a/iov.h
+++ b/iov.h
@@ -19,7 +19,7 @@
#include <string.h>
size_t iov_skip_bytes(const struct iovec *iov, size_t n,
- size_t vec_offset, size_t *buf_offset);
+ size_t skip, size_t *offset);
size_t iov_from_buf(const struct iovec *iov, size_t iov_cnt,
size_t offset, const void *buf, size_t bytes);
size_t iov_to_buf(const struct iovec *iov, size_t iov_cnt,