Re: [PATCH v4 3/8] status: support --porcelain[=]

2016-08-03 Thread Junio C Hamano
Jeff Hostetler writes: > + else if (!strcmp(arg, "v1") || !strcmp(arg,"1")) Missing SP between arg, and "1" here (FYI, editing it here to fix would affect a later patch that has this line in the context). Other than that looking good so far. -- To unsubscribe from this list: send the line "

[PATCH v4 3/8] status: support --porcelain[=]

2016-08-02 Thread Jeff Hostetler
From: Jeff Hostetler Update --porcelain argument to take optional version parameter to allow multiple porcelain formats to be supported in the future. The token "v1" is the default value and indicates the traditional porcelain format. (The token "1" is an alias for that.) Signed-off-by: Jeff H