Ping - is there any feedback on this trivial change?
On Mon, 26 Oct 2020 22:03:10 +0100, David Disseldorp wrote:
> sg_copy_buffer() returns a size_t with the number of bytes copied.
> Return 0 instead of false if the copy is skipped.
>
> Signed-off-by: David Disseldorp <ddiss@xxxxxxx>
> Reviewed-by: Douglas Gilbert <dgilbert@xxxxxxxxxxxx>
> ---
> lib/scatterlist.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/lib/scatterlist.c b/lib/scatterlist.c
> index 0a482ef988e5..a59778946404 100644
> --- a/lib/scatterlist.c
> +++ b/lib/scatterlist.c
> @@ -933,7 +933,7 @@ size_t sg_copy_buffer(struct scatterlist *sgl, unsigned int nents, void *buf,
> sg_miter_start(&miter, sgl, nents, sg_flags);
>
> if (!sg_miter_skip(&miter, skip))
> - return false;
> + return 0;
>
> while ((offset < buflen) && sg_miter_next(&miter)) {
> unsigned int len;
