From aa685310bb49e9d1ba99a254a63293bf9236838c Mon Sep 17 00:00:00 2001 From: Sergio Costas Date: Sun, 14 Oct 2018 14:05:04 +0200 Subject: [PATCH] Fix enumeration in HACKING.md The HACKING.md file contains an enumeration in the Indentation and whitespace section, but the sentences look incorrectly divided. This patch proposes a fix for that section, reordering all its contents in a single enumeration, instead of a paragraph and an enumeration, and also reconstruct the sentences. --- HACKING.md | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/HACKING.md b/HACKING.md index 8c064999a..0b86763da 100644 --- a/HACKING.md +++ b/HACKING.md @@ -10,15 +10,15 @@ Life isn't fun if you can't break the rules. If a rule seems unnecessarily restrictive while you're coding, ignore it, and let the patch reviewer decide what to do. -## Indentation and whitespace +## Indentation, braces and whitespace -Use four-space indents. Braces are on the same line as their associated -statements. You should only omit braces if *both* sides of the statement are -on one line. +* Use four-space indents. +* Braces are on the same line as their associated statements. +* You should only omit braces if *both* sides of the statement are on one line. +* One space after the `function` keyword. +* No space between the function name in a declaration or a call. +* One space before the parens in the `if` statements, or `while`, or `for` loops. -* One space after the `function` keyword. No space between the function name -* in a declaration or a call. One space before the parens in the `if` -* statements, or `while`, or `for` loops. ```javascript function foo(a, b) { let bar;