On Monday, 07 January 2008, at 19:58:38 (+0100), Kim Woelders wrote: > I put that file there at a point where I got annoyed with the > formatting style mess in imlib2. The formatting style details, among > other things tabs/spaces, were set up in an attempt to minimize the > overall change caused by running indent on src/lib/*.c at that > time. IIRC using spaces in stead of tabs resulted in slightly fewer > changes. > > Actually, I think the "original raster formatting style" is closer > to the one in e16/e/.indent.pro, which specifies to use tabs. > > I think it would be a good thing to specify a recommended/default > indent profile, and require that apps/libs explicitly place a > .indent.pro at top level if they want to deviate.
Here are the original indent profiles from back when we first started using indent: alias indent-mej="indent -bap -bbo -br -brs -ce -ci4 -cli2 -cs -di1 -hnl -i4 -l140 -lp -lps -nbc -npcs -nss -nsob -pmt -psl -ts 999" alias indent-raster="indent -i3 -bl -bad -nbap -sob -ncdb -di20 -nbc -lp -nce -npcs -sc -ncs -l80" alias indent-mandrake="indent -i3 -bl -bad -nbap -sob -ncdb -di20 -nbc -lp -nce -npcs -sc -ncs -l80" alias indent-gilbertt="indent -nbad -bap -nbc -bbo -bl -bli2 -bls -ncdb -nce -cp1 -cs -di2 -ndj -nfc1 -nfca -hnl -i2 -ip5 -lp -pcs -psl -nsc -nsob" alias indent-ck="indent -i3 -bl -bad -nbap -sob -ncdb -di20 -nbc -lp -nce -npcs -sc -ncs -l80" At this point in time, I'm pretty sure we (or at least I) looked for a way to prevent the use of tabs, and there was no option for it at the time (at least that we found) in the version we were using. You'll notice the "-ts 999" in mine -- that did essentially the same thing. This was back in 1999, so things have changed since then, obviously. Bottom line: Spaces make sure everyone sees the same indentation. Tabs do not. Since re-indenting code is very easy these days, using tabs as a way of defining spacing is no longer useful and actually does more harm than good. So use spaces, not tabs. If your version of indent still defaults to using tabs, add "-nut". Michael -- Michael Jennings (a.k.a. KainX) http://www.kainx.org/ <[EMAIL PROTECTED]> Linux Server/Cluster Admin, LBL.gov Author, Eterm (www.eterm.org) ----------------------------------------------------------------------- "And you women try to make us feel better about it. 'It's not the size of the ship that counts; it's the motion of the ocean.' Well, that may be true, but it takes an awfully long time to get to England in a rowboat." -- Jeff Foxworthy ------------------------------------------------------------------------- Check out the new SourceForge.net Marketplace. It's the best place to buy or sell services for just about anything Open Source. http://ad.doubleclick.net/clk;164216239;13503038;w?http://sf.net/marketplace _______________________________________________ enlightenment-devel mailing list enlightenment-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
