Subject: Re: [PATCH] string_list: use string-list API in unsorted_string_list_lookup()

From: Ralf Thielow <[email-masked]>
Date: Mon, 25 Apr 2016 19:35:41 +0200

Johannes Schindelin <[email-masked]> writes:

>>  
>> -	for (i = 0; i < list->nr; i++)
>> -		if (!cmp(string, list->items[i].string))
>> -			return list->items + i;
>> +	for_each_string_list_item(item, list) {
>> +		if (!cmp(string, item->string))
>> +			return item;
>> +	}
>>  	return NULL;
>
> If you drop the extra curly braces (thereby matching Git's coding style as
> an additional bonus), at least the patch won't increase the number of
> lines.
>

Oops. I'll send a new version of the patch without these
extra curly braces.

Thanks!

> In any case, I like the simplification of the code.
>
> Ciao,
> Dscho