Stepan, Sorry. I didn't mean to blame you. I did realise that you were just putting it back to the "initial" style.
Since you asked (well almost ;-) I'm in favour of 4 space indent since this is quite common in the java code. Unlike Geir I don't mind if tabs are used but they should always be treated as being equivalent to 8 spaces. Though I'd might change my mind if I understood the motivation behind Geir's aversion to tabs. Regards, Mark. On 4/18/06, Stepan Mishura <[EMAIL PROTECTED]> wrote: > Mark, > > The update only adjusts build files to *initial* style (that is tabs-style). > > IMHO really doesn't matter how many spaces in one tab if you follow one > style in a file and don't mix tabs with spaces. Also I like > tab-style because I have opportunity to choose a number of spaces that more > suitable for my eyes. If everybody will agree to use only space-style then > OK - I'll fix tabs. > > Thanks, > Stepan. > > > On 4/18/06, Mark Hindess wrote: > > > > For many people however tabs are equivalent to 8 spaces so if what we > > really intend is 4 spaces then perhaps we should make them spaces not > > tabs? > > > > Otherwise we will be forever fixing identations because of editor > > differences. > > > > -Mark. > > > > On 4/18/06, Stepan Mishura <[EMAIL PROTECTED]> wrote: > > > On 4/18/06, Mark Hindess wrote: > > > > > > > > Stepan, > > > > > > > > Just curious what you are fixing here? Changing 8 spaces to tabs? > > > > Why does this matter? Shouldn't tabs (at the beginning of a line) > > > > always be equivalent to 8 spaces? > > > > > > > > > Mark, > > > > > > I'd prefer to have all build files follow one style - I don't like > > mixing > > > tabs and spaces even they looks the same. For this particular case - > > build > > > files were initially created using tabs and I'd prefer to keep this > > style. > > > Also for me tab is not equivalent to 8 spaces - Eclipse sets tab > > equivalent > > > to 4 spaces and I'm not going to change it because I like it :-) > > > > > > Thanks, > > > Stepan. > > > > > > Incidentally, I think 8 character indentations are excessive. Quite a > > > > few of the ant files use 4 character indentations which I find much > > > > easier to read. Ditto for java code. Perhaps we could agree which to > > > > use? > > > > > > > > Regards, > > > > -Mark - wondering if he might regret asking this > > > > > > > > On 4/18/06, [EMAIL PROTECTED] <[EMAIL PROTECTED]> wrote: > > > > > Author: smishura > > > > > Date: Tue Apr 18 02:38:29 2006 > > > > > New Revision: 394890 > > > > > > > > > > URL: http://svn.apache.org/viewcvs?rev=394890&view=rev > > > > > Log: > > > > > Correcting indentation > > > > > > > > > > Modified: > > > > > > > > > > > incubator/harmony/enhanced/classlib/trunk/modules/applet/make/common/build.xml > > > > > > > > > > > incubator/harmony/enhanced/classlib/trunk/modules/archive/make/common/build.xml > > > > > > > > > > > incubator/harmony/enhanced/classlib/trunk/modules/auth/make/common/build.xml > > > > > > > > > > > incubator/harmony/enhanced/classlib/trunk/modules/awt/make/common/build.xml > > > > > > > > > > > incubator/harmony/enhanced/classlib/trunk/modules/beans/make/common/build.xml > > > > > > > > > > > incubator/harmony/enhanced/classlib/trunk/modules/crypto/make/common/build.xml > > > > > > > > > > > incubator/harmony/enhanced/classlib/trunk/modules/jndi/make/common/build.xml > > > > > > > > > > > incubator/harmony/enhanced/classlib/trunk/modules/logging/make/common/build.xml > > > > > > > > > > > incubator/harmony/enhanced/classlib/trunk/modules/luni/make/common/build.xml > > > > > > > > > > > incubator/harmony/enhanced/classlib/trunk/modules/math/make/common/build.xml > > > > > > > > > > > incubator/harmony/enhanced/classlib/trunk/modules/nio/make/common/build.xml > > > > > > > > > > > incubator/harmony/enhanced/classlib/trunk/modules/nio_char/make/common/build.xml > > > > > > > > > > > incubator/harmony/enhanced/classlib/trunk/modules/prefs/make/common/build.xml > > > > > > > > > > > incubator/harmony/enhanced/classlib/trunk/modules/regex/make/common/build.xml > > > > > > > > > > > incubator/harmony/enhanced/classlib/trunk/modules/rmi/make/common/build.xml > > > > > > > > > > > incubator/harmony/enhanced/classlib/trunk/modules/security/make/common/build.xml > > > > > > > > > > > incubator/harmony/enhanced/classlib/trunk/modules/sql/make/common/build.xml > > > > > > > > > > > incubator/harmony/enhanced/classlib/trunk/modules/text/make/common/build.xml > > > > > > > > > > > incubator/harmony/enhanced/classlib/trunk/modules/x-net/make/common/build.xml > > > > > > > > > > Modified: > > > > > > incubator/harmony/enhanced/classlib/trunk/modules/applet/make/common/build.xml > > > > > URL: > > > > > > http://svn.apache.org/viewcvs/incubator/harmony/enhanced/classlib/trunk/modules/applet/make/common/build.xml?rev=394890&r1=394889&r2=394890&view=diff > > > > > > > > > > > ============================================================================== > > > > > --- > > > > > > incubator/harmony/enhanced/classlib/trunk/modules/applet/make/common/build.xml > > > > (original) > > > > > +++ > > > > > > incubator/harmony/enhanced/classlib/trunk/modules/applet/make/common/build.xml > > > > Tue Apr 18 02:38:29 2006 > > > > > @@ -68,12 +68,12 @@ > > > > > > > > > > > > > > > <target name="run.tests"> > > > > > - > > > > > - <mkdir dir="${hy.tests.reports}" /> > > > > > + > > > > > + <mkdir dir="${hy.tests.reports}" /> > > > > > > > > > > <property name="test.jre.home" value="${hy.target > > }/jre" > > > > /> > > > > > > > > > > - <junit fork="yes" > > > > > + <junit fork="yes" > > > > > forkmode="once" > > > > > printsummary="withOutAndErr" > > > > > errorproperty="test.errors" > > > > > > > > > > [ SNIP ] > > > > > > > > -- > > > > Mark Hindess <[EMAIL PROTECTED]> > > > > IBM Java Technology Centre, UK. > > > > > > > > --------------------------------------------------------------------- > > > > Terms of use : http://incubator.apache.org/harmony/mailing.html > > > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > > > For additional commands, e-mail: [EMAIL PROTECTED] > > > > > > > > > > > > > > > > > -- > > > ----------------------------------------------------------- > > > Terms of use : http://incubator.apache.org/harmony/mailing.html > > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > > For additional commands, e-mail: [EMAIL PROTECTED] > > > > > > Thanks, > > > Stepan Mishura > > > Intel Middleware Products Division > > > > > > > > > > > > -- > > Mark Hindess <[EMAIL PROTECTED]> > > IBM Java Technology Centre, UK. > > > > --------------------------------------------------------------------- > > Terms of use : http://incubator.apache.org/harmony/mailing.html > > To unsubscribe, e-mail: [EMAIL PROTECTED] > > For additional commands, e-mail: [EMAIL PROTECTED] > > > > > > > -- > ----------------------------------------------------------- > Terms of use : http://incubator.apache.org/harmony/mailing.html > To unsubscribe, e-mail: [EMAIL PROTECTED] > For additional commands, e-mail: [EMAIL PROTECTED] > > Thanks, > Stepan Mishura > Intel Middleware Products Division > > -- Mark Hindess <[EMAIL PROTECTED]> IBM Java Technology Centre, UK. --------------------------------------------------------------------- Terms of use : http://incubator.apache.org/harmony/mailing.html To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]