On Tue, Sep 06, 2011 at 12:13:17PM -0700, Mark Diggory wrote: > Correct, TBH, The critical detail of our standard were not JAVA formatting > specific. I rather care less about where curly brackets are/arn't. > > The most important feature of the original requirements was that > "whitespace" be standardized to eliminate windows linefeeds getting into > sourcecode and creating commits of code that have no real semantic change.
The way to fix that is to set the svn:eol-style=native on all text files, to ask developers to do likewise on new text files, and to point developers to ways of making this automatic in various environments. Then every checkout will set those files up properly for the user checking them out, and files in the SCM will remain normalized. > Code formatting in Eclipse/IDEA using standard defaults is best. But > historically, our codebase has a mixing of formatting caused by past commits > by developers using terminal clients not wanting to bother with proper > formatting. So our own code doesn't actually adhere wholly to our own > current formatting standards and TBH, going through and reformatting those > sections (or the entire codebase to make it fit the default standard) would > result in such a large number of "non-semantic" line changes, I'm not sure > the benefit outweighs the "merge nightmare" for users that it would create. As you may have noticed, I tend to just tidy up formatting (and spelling) whenever it gets in my way. I'd like to believe that this gradually makes things more regular overall. -- Mark H. Wood, Lead System Programmer [email protected] Asking whether markets are efficient is like asking whether people are smart.
pgpnwuggT8NJR.pgp
Description: PGP signature
------------------------------------------------------------------------------ Using storage to extend the benefits of virtualization and iSCSI Virtualization increases hardware utilization and delivers a new level of agility. Learn what those decisions are and how to modernize your storage and backup environments for virtualization. http://www.accelacomm.com/jaw/sfnl/114/51434361/
_______________________________________________ Dspace-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/dspace-devel
