On Oct 28, 2007, at 8:28 AM, Junio C Hamano wrote:

quoted text
> Steffen Prohaska <prohaska@zib.de> writes:
>
>> This teaches "push <remote> HEAD" to resolve HEAD on the local
>> side to its real ref name, e.g. refs/heads/master, and then
>> use the real ref name on the remote side to search a matching
>> remote ref.
>
> This probably is a good idea.

I'll think about Daniel's suggestion of implementing it
differently.


quoted text
>> +	if (real_name) {
>> +		free(real_name);
>> +	}
>
> Excess and unnecessary brace pair.

When are excess and unnecessary braces acceptable?
Is

	if (something) {
  		printf("text %d"
   		       "more text %d"
                        "and even more %d\n"
		       , a, b, c);
	}

ok? Or should I avoid braces there, too?

	Steffen
-
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html