Thread (1 message) 1 message, 1 author, 2016-06-15

Re: [PATCH v10 09/14] git-clean: use a git-add-interactive compatible UI

From: Junio C Hamano <hidden>
Date: 2016-06-15 22:57:17

Junio C Hamano [off-list ref] writes:
Also please fix this one:
quoted
+		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".
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help