On 08/29/2013 02:34 PM, Junio C Hamano wrote:
> Brad King <brad.king@xxxxxxxxxxx> writes:
>
>> + const char *c, *s, *oldvalue, *value[2] = {0,0};
>
> This patch has many style issues of the same kind, lack of a SP at
> places where there should be between operators and after comma.
Okay, I can fix those. However, for this patch I'm particularly
interested in suggestions for the proposed stdin format. Right
now it just looks like the command line, but it feels strange to
parse "-options" from a formatted input stream that is not an
options response file.
Thanks,
-Brad
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html