Difference between revisions of "Preparing a Patch for Review"
| Line 1: | Line 1: | ||
<h2>Best Practices</h2> | <h2>Best Practices</h2> | ||
| + | |||
| + | Avoid saving tab characters in the patch. The main reason is that different text editors use different defaults to interpret tab characters as spaces. Also some people like to set 8 spaces as the default for a tab character while others prefer 4 or 3. The | ||
| + | end result is that the proposed code will look unformatted, making hard to the eye to read and follow the code indentations. | ||
Revision as of 21:15, 30 August 2012
Best Practices
Avoid saving tab characters in the patch. The main reason is that different text editors use different defaults to interpret tab characters as spaces. Also some people like to set 8 spaces as the default for a tab character while others prefer 4 or 3. The
end result is that the proposed code will look unformatted, making hard to the eye to read and follow the code indentations.
