lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Erik Hatcher <>
Subject Re: tabs in .java files
Date Fri, 26 Mar 2004 01:25:35 GMT
One more comment on this topic.... thanks to being involved in Lucene, 
I have a much deeper respect for developer individuality in code 
formatting.  In fact, 4-space indents seem excessive to me now after 
being deep into the Lucene code (not to mention writing articles/books, 
where indents need to be smaller).

I still hate tabs though.  Invisible little monsters, they are.


On Mar 25, 2004, at 5:18 PM, Doug Cutting wrote:

> -1
> They'll just creep back in.  Plus I generally think reformatting is a 
> bad reason for a commit.  It clogs and confuses diffs.  Changes should 
> be confined to fixing bugs and adding features.  To me, reformatting 
> is more evil than tabs.
> Tab's are acceptable so long as things indent correctly with 8 space 
> stops.
> Doug
> Otis Gospodnetic wrote:
>> Tabs are evil.
>> Does anyone mind me doing s/\t/  /g on *.java?
>> I'd love to clean up, but am wondering if that will affect our ability
>> to apply patches sitting in Bugzilla.
>> Does anyone have any experience with this?
>> A quick glance at the man page for patch seems to indicate that this
>> should not be a problem:
>> -l  or  --ignore-whitespace
>> Match  patterns  loosely, in case tabs or spaces have been munged in
>> your files.  Any sequence of one or more blanks in  the  patch  file
>> matches  any  sequence in the original file, and sequences of blanks
>> at the ends of lines are  ignored.   Normal  characters  must  still
>> match  exactly.  Each line of the context must still match a line in
>> the original file.
>> I'll commit my local s/\t/  /g changes, unless somebody speaks up by
>> tomorrow morning, CET.
>> Thanks,
>> Otis
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail:
>> For additional commands, e-mail:
> ---------------------------------------------------------------------
> To unsubscribe, e-mail:
> For additional commands, e-mail:

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message