On Thu, Jan 8, 2009 at 12:43, Nick Coghlan <ncoghlan at gmail.com> wrote: > Brett Cannon wrote: >> Can we then all agree that a policy of re-indenting per function as >> changes are made to the code is acceptable but not required? > > Such a rule would certainly make *my* life a lot easier - the reason I > find the tabs annoying is because I have my editor set to switch > everything to 4 space indents by default, and I have to fiddle with it > to get it to keep the tabs when I'm editing functions/files that > previously used tabs for indenting. > > Even if we do adopt such a rule, C patches posted to the tracker should > still try to avoid including pure whitespace changes though - leaving > the whitespace changes in the patch tends to lead to patches that look > like "remove function body, add different function body" when only a > couple of lines have actually had significant changes. > That's fine with me. Correcting whitespace can be considered a committer's job. -Brett
RetroSearch is an open source project built by @garambo | Open a GitHub Issue
Search and Browse the WWW like it's 1997 | Search results from DuckDuckGo
HTML:
3.2
| Encoding:
UTF-8
| Version:
0.7.4