From: Junio C Hamano <gitster@pobox.com>
Date: Thu, 16 May 2013 19:39:47 GMT
Subject: Re: [PATCH v10 09/14] git-clean: use a git-add-interactive compatible UI
Message-ID: <7vtxm24su4.fsf@alter.siamese.dyndns.org>
In-Reply-To: <7vy5be4tbf.fsf@alter.siamese.dyndns.org>

Junio C Hamano <gitster@pobox.com> writes:

> Also please fix this one:
>
>> +		for_each_string_list_item(item, (struct string_list *)stuff->stuff) {
>> +			if ((*chosen)[i] < 0)
>> +				(*chosen)[i] = 0;
>> +			strbuf_addf(&menu, "%s%2d: %s", (*chosen)[i] ? "*" : " ", ++i, item->string);
>
> Because the evaluation order of function arguments are not defined
> (not left to right; these are comma-expressions),

I cannot spell, sorry.  s/are not defined/is not defined/.  Also
s/these are c/these are not c/;

>
> 	(*chosen)[i] ? "*" : " "
>
> may use the original value of "i", or value after increment the
> evaluation of
>
> 	++i
>
> left in "i".