Junio C Hamano <gitster@xxxxxxxxx> writes: > 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. That's not grammatical. -- David Kastrup -- 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