Skip to content

Commit dd30800

Browse files
committed
CodingGuidelines: once it is in, it is not worth the code churn
Signed-off-by: Junio C Hamano <[email protected]>
1 parent 7e76a2f commit dd30800

File tree

1 file changed

+8
-0
lines changed

1 file changed

+8
-0
lines changed

Documentation/CodingGuidelines

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,14 @@ code. For Git in general, three rough rules are:
1818
judgement call, the decision based more on real world
1919
constraints people face than what the paper standard says.
2020

21+
- Fixing style violations while working on a real change as a
22+
preparatory clean-up step is good, but otherwise avoid useless code
23+
churn for the sake of conforming to the style.
24+
25+
"Once it _is_ in the tree, it's not really worth the patch noise to
26+
go and fix it up."
27+
Cf. http://article.gmane.org/gmane.linux.kernel/943020
28+
2129
Make your code readable and sensible, and don't try to be clever.
2230

2331
As for more concrete guidelines, just imitate the existing code

0 commit comments

Comments
 (0)