On a related note, I wrote a few scripts to detect and correct some types of such style errors, see llvm/utils/lint/* .<br>I also added a function to llvm/utils/vim/vimrc to delete trailing whitespace and highlight existing trailing whitespace -- if anyone's an Emacs-lisp hacker, please add it to the emacs config file as well.<br>
<br>Sure, this doesn't enforce anything, but I'm hoping folks will start to use these tools and will over time clean up the style in the entire code base.<br><br><div class="gmail_quote">2009/2/20 Scott Michel <span dir="ltr"><<a href="mailto:scooter.phd@gmail.com">scooter.phd@gmail.com</a>></span><br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">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.</blockquote>
<div><br>I've been fixing things on a directory-by-directory basis as I come across style violations while browsing the code.  I'm not in favor of a single global change to fix everything everywhere; I think this can be done gradually over time and the diff will be easier to read if it's smaller, so you can verify that the script (or your editor) did not mangle anything.<br>
<br>Misha<br></div></div>