The point immediately before it is about having SP after the control keyword. Follow it. Signed-off-by: Junio C Hamano <gitster@xxxxxxxxx> --- Documentation/CodingGuidelines | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/CodingGuidelines b/Documentation/CodingGuidelines index 1b0cd2b..21e4272 100644 --- a/Documentation/CodingGuidelines +++ b/Documentation/CodingGuidelines @@ -184,7 +184,7 @@ For C programs: of "else if" statements, it can make sense to add braces to single line blocks. - - We try to avoid assignments inside if(). + - We try to avoid assignments inside "if ()" condition. - Try to make your code understandable. You may put comments in, but comments invariably tend to stale out when the code -- 2.0.0-rc1-355-gd6d6511 -- 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