On Thu, 25 Sep 2003, Dave Airlie wrote:
>
> If Linus thinks your patch goes far enough for him..
I actually usually don't much like whitespace changes, and the one I sent
was purely in response to the fact that there already was a lot of
whitespace changes in the i810 driver.
Don't worry too much about it. The i810 whitespace fixes were fine - it just
became obvious that when they fixed something like
if(x)
to
if (x)
they still left strange mixes of spaces and tabs on the same line.
All my whitespace changes was _literally_ from just editing the DRI CVS
merge diff (any lines added) before applying to the kernel tree.
I'll happily take more whitespace fixes, I just ask that as long as you
play around with whitespace you don't do any other changes. A big
whitespace diff that also has a subtle semantic change embedded in it is
just too painful.
Linus
-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf
_______________________________________________
Dri-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/dri-devel
