For the complete truth in advertising, this was pretty much a trial balloon to gauge reaction. I'm not a big fan of rejecting commits for style violations, but the dev guide has certain guidelines regarding formatting and style. And we're all supposed to be good citizens...<br>
<br>My biggest nit, however, was contemplating a commit where 80%+ was trailing whitespace trimming. Yeah, my editor happens to practice good hygiene. I could have been a complete *$$**le and committed a global hygiene patch. I only touched the files that I'll end up committing in another day or two.<br>
<br>Since there are style rules, I also decided to see how much reaction there would be if they were enforced at commit time. Evidently, it's as popular as a skunk at a garden party. So, it's not something I'm personally looking to invest much time into.<br>
<br><br>-scooter<br><br><div class="gmail_quote">On Wed, Feb 18, 2009 at 8:53 AM, Devang Patel <span dir="ltr"><<a href="mailto:dpatel@apple.com">dpatel@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="Ih2E3d"><br>
On Feb 18, 2009, at 1:41 AM, Julien Lerouge wrote:<br>
<br>
> Yet another _fun_ way of doing this is to setup a buildbot slave just<br>
> for that. The slave can fix minor stuff like tabs and trailing<br>
> whitespaces on its own (checking the changes back in), and yell for<br>
> things like 80-col violations and whatnot where the changes would<br>
> not be<br>
> so trivial.<br>
<br>
</div>If you're going to change anything then this is the best alternative,<br>
otherwise I can live with status quo.<br>
<br>
Do not reject commit just because of formatting issues. It can have<br>
serious -ve impact on productivity.<br>
<br>
To folks who prefers to reject commits due to formatting errors -- You<br>
already rely on a some kind of "tool" to make your day to day life<br>
easier. [ Most likely you've your editor automatically replacing tabs<br>
into spaces. Your terminal window is only 80 col. wide or your editor<br>
is displaying a vertical line to warn you about 80 col. and so<br>
on... ]. The  build bot suggested by Julien is yet another "tool" that<br>
accomplishes the same.  One the slave bot can use clang static<br>
analyzer ... :)<br>
<br>
-<br>
<font color="#888888">Devang<br>
</font><div><div></div><div class="Wj3C7c">_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:LLVMdev@cs.uiuc.edu">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu" target="_blank">http://llvm.cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
</div></div></blockquote></div><br>