I think I will start with what we have now by just switching the curls to the same line.. Looking at our checkstyle they are not as strict as CXF (the one mentioned here).
We can improve later if needed without any problems. On Tue, Aug 4, 2015 at 11:28 AM, Daniel Kulp <[email protected]> wrote: > Lazy consensus is fine. Wait till tomorrow and if no one objects, just do > it. > > Dan > > >> On Aug 4, 2015, at 11:24 AM, Clebert Suconic <[email protected]> >> wrote: >> >> Is this a just do it thing, or we need a vote? >> >> On Tue, Aug 4, 2015 at 10:07 AM, Timothy Bish <[email protected]> wrote: >>> On 08/04/2015 07:56 AM, Christopher Shannon wrote: >>>> So in general I'm not too picky with coding styles but I just started >>>> looking at the Artemis project a few days ago and something that stood out >>>> to me right away was the use of opening curly braces on a new line. >>>> >>>> Virtually ever Java code base I've seen is written in the style using the >>>> opening brace on the same line. (See >>>> http://google.github.io/styleguide/javaguide.html#s4.1.2-blocks-k-r-style) >>>> I think that in general it would be a good idea to match up to a style >>>> that most Java developers are used to working with if we want to get more >>>> of the community involved. >>>> >>>> I was wondering if anyone would have an issue with changing the style or >>>> what people's thoughts are about this potential change? >>>> >>> +1 >>> >>> I think following the javaguide style would make the code much more >>> approachable. >>> >>> -- >>> Tim Bish >>> Sr Software Engineer | RedHat Inc. >>> [email protected] | www.redhat.com >>> twitter: @tabish121 >>> blog: http://timbish.blogspot.com/ >>> >> >> >> >> -- >> Clebert Suconic > > -- > Daniel Kulp > [email protected] - http://dankulp.com/blog > Talend Community Coder - http://coders.talend.com > -- Clebert Suconic
