Re: [crypto] The standard indentation is 4 spaces per indent

2016-04-29 Thread Gangumalla, Uma
>This is Commons, AND this is brand new code, so in my mind there is no "original" formatting style to respect. 4 spaces per indent like the rest of the project please. Remember that Commons is a SINGLE project. Hopefully we won't have to argue about too much about this... [Uma] Thanks Gary for

RE: [crypto] The standard indentation is 4 spaces per indent

2016-04-29 Thread Chen, Haifeng
Thanks, > > Haifeng > > > > -Original Message- > > From: Matt Sicker [mailto:boa...@gmail.com] > > Sent: Friday, April 29, 2016 9:00 AM > > To: Commons Developers List <dev@commons.apache.org> > > Subject: Re: [crypto] The standard indentation i

Re: [crypto] The standard indentation is 4 spaces per indent

2016-04-28 Thread Gary Gregory
gt; > > > -Original Message- > > From: Matt Sicker [mailto:boa...@gmail.com] > > Sent: Friday, April 29, 2016 9:00 AM > > To: Commons Developers List <dev@commons.apache.org> > > Subject: Re: [crypto] The standard indentation is 4 spaces per indent > >

Re: [crypto] The standard indentation is 4 spaces per indent

2016-04-28 Thread Dan Tran
mmons Developers List <dev@commons.apache.org> > Subject: Re: [crypto] The standard indentation is 4 spaces per indent > > If you want to prevent mixed whitespace styles and whatnot, you can use > EditorConfig <http://editorconfig.org/>. > > On 28 April 2016 at 19:06, Ga

RE: [crypto] The standard indentation is 4 spaces per indent

2016-04-28 Thread Chen, Haifeng
] Sent: Friday, April 29, 2016 9:00 AM To: Commons Developers List <dev@commons.apache.org> Subject: Re: [crypto] The standard indentation is 4 spaces per indent If you want to prevent mixed whitespace styles and whatnot, you can use EditorConfig <http://editorconfig.org/>. On 28 Apri

Re: [crypto] The standard indentation is 4 spaces per indent

2016-04-28 Thread Matt Sicker
rd indentation is > >>>>4 > >> spaces per indent - but respect the number of spaces used by the > >>original." > >>>>The [crypto] Java code I've seen to far is all 2 spaces per indent. > >>>>I think now is the time to do this,

Re: [crypto] The standard indentation is 4 spaces per indent

2016-04-28 Thread Gangumalla, Uma
be co-ordinated with other changes. > >So I think it would be wrong to commit the change without some prior >notification. >This can either be a JIRA or agreement on the dev list. > >> Regards, >> Haifeng >> >> -Original Message- >> From: Gary G

Re: [crypto] The standard indentation is 4 spaces per indent

2016-04-26 Thread sebb
> -Original Message- > From: Gary Gregory [mailto:garydgreg...@gmail.com] > Sent: Tuesday, April 26, 2016 9:53 AM > To: Commons Developers List <dev@commons.apache.org> > Subject: RE: [crypto] The standard indentation is 4 spaces per indent > > Hi, > > Do you

RE: [crypto] The standard indentation is 4 spaces per indent

2016-04-25 Thread Chen, Haifeng
: Commons Developers List <dev@commons.apache.org> Subject: RE: [crypto] The standard indentation is 4 spaces per indent Hi, Do you really want this level of Jira tracking? It seems over the top to me. Is this process style for this component? In this case I would just do it and not Jira it. Then

RE: [crypto] The standard indentation is 4 spaces per indent

2016-04-25 Thread Chen, Haifeng
] Sent: Tuesday, April 26, 2016 6:25 AM To: Commons Developers List <dev@commons.apache.org> Subject: [crypto] The standard indentation is 4 spaces per indent Hi all, In our coding guidelines [1] we say that "The standard indentation is 4 spaces per indent - but respect the number of spaces

[crypto] The standard indentation is 4 spaces per indent

2016-04-25 Thread Gary Gregory
Hi all, In our coding guidelines [1] we say that "The standard indentation is 4 spaces per indent - but respect the number of spaces used by the original." The [crypto] Java code I've seen to far is all 2 spaces per indent. I think now is the time to do this, most IDEs can do a one-shot format