Re: [PATCH v3 01/27] alist: add a helper to check if the list is full

[Simon Glass]

Hi Sughosh,

On Wed, 21 Aug 2024 at 04:59, Sughosh Ganu <sughosh.g...@linaro.org> wrote:
>
> Add a helper function to check if the alist is full. This can then be
> used to extend the alist.
>
> Signed-off-by: Sughosh Ganu <sughosh.g...@linaro.org>
> ---
> Changes since V2: None
>
>  include/alist.h | 11 +++++++++++
>  1 file changed, 11 insertions(+)

Reviewed-by: Simon Glass <s...@chromium.org>

I would still rather add an alist_insert() function, but we can clean
that up later.

>
> diff --git a/include/alist.h b/include/alist.h
> index 586a1efa5c..68d268f01a 100644
> --- a/include/alist.h
> +++ b/include/alist.h
> @@ -82,6 +82,17 @@ static inline bool alist_err(struct alist *lst)
>         return lst->flags & ALISTF_FAIL;
>  }
>
> +/**
> + * alist_full() - Check if the alist is full
> + *
> + * @lst: List to check
> + * Return: true if full, false otherwise
> + */
> +static inline bool alist_full(struct alist *lst)
> +{
> +       return lst->count == lst->alloc;
> +}
> +
>  /**
>   * alist_get_ptr() - Get the value of a pointer
>   *
> --
> 2.34.1
>

Regards,
Simon