+1 I don't like the curly brace on new line rule as it wastes vertical space !
On 2 May 2014 14:57, Gary Gregory <garydgreg...@gmail.com> wrote: > +1 > > Gary > > > On Fri, May 2, 2014 at 6:53 AM, Benedikt Ritter <brit...@apache.org> wrote: > >> Hi all, >> >> BeanUtils2 uses the maven code style rules. Every now and then I have to >> reject patches because they don't follow these conventions. I'd like to >> change the formatting rules to a more standard rule set for the following >> reasons: >> >> - maven style is very verbose, since it uses a lot of spaces and curly >> braces have to be placed on new lines >> - a great majority of our components uses the Sun/Eclipse formatting with >> spaces >> - Setting up IDEs like IntelliJ or Eclipse is easier with the Sun/Eclipse >> style, because the only thing you have to change is the tab policy. Using >> the maven style implies that one has to import the rule set. >> >> If no one objects, I'll reformat the source code and update the check style >> rules. >> >> Regards, >> Benedikt >> >> >> -- >> http://people.apache.org/~britter/ >> http://www.systemoutprintln.de/ >> http://twitter.com/BenediktRitter >> http://github.com/britter >> > > > > -- > E-Mail: garydgreg...@gmail.com | ggreg...@apache.org > Java Persistence with Hibernate, Second > Edition<http://www.manning.com/bauer3/> > JUnit in Action, Second Edition <http://www.manning.com/tahchiev/> > Spring Batch in Action <http://www.manning.com/templier/> > Blog: http://garygregory.wordpress.com > Home: http://garygregory.com/ > Tweet! http://twitter.com/GaryGregory --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org For additional commands, e-mail: dev-h...@commons.apache.org