From: Miklos Vajna Date: Thu, 22 May 2008 23:26:09 GMT Subject: [PATCH] CodingGuidelines: Add a note to avoid assignments inside if() Message-ID: <1211498769-26871-1-git-send-email-vmiklos@frugalware.org> Signed-off-by: Miklos Vajna --- I just had to ask Dscho about this. Better if it's documented, I suppose. Documentation/CodingGuidelines | 2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/Documentation/CodingGuidelines b/Documentation/CodingGuidelines index 994eb91..d2a0a76 100644 --- a/Documentation/CodingGuidelines +++ b/Documentation/CodingGuidelines @@ -89,6 +89,8 @@ 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(). + - Try to make your code understandable. You may put comments in, but comments invariably tend to stale out when the code they were describing changes. Often splitting a function -- 1.5.5.1.357.g1af8b.dirty