Subject: Re: [PATCH v2 6/7] string-list: optionally omit empty string pieces in string_list_split*()

From: shejialuo <[email-masked]>
Date: Fri, 1 Aug 2025 11:33:50 +0800

On Thu, Jul 31, 2025 at 03:46:05PM -0700, Junio C Hamano wrote:
> diff --git a/string-list.c b/string-list.c
> index 86a309f8fb..343cf1ca90 100644
> --- a/string-list.c
> +++ b/string-list.c
> @@ -294,6 +294,9 @@ static int append_one(struct string_list *list,
>  				break;
>  	}
>  
> +	if ((flags & STRING_LIST_SPLIT_NONEMPTY) && (end <= p))
> +		return 0;
> +

I somehow think we should do this directly in `split_string` function.
And should we use `end == p`?

Thanks,
Jialuo