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]
