Messages in this thread |  | | Date | Thu, 29 Nov 2001 16:02:18 +0000 | From | Padraig Brady <> | Subject | Re: Extraneous whitespace removal? |
| |
Wayne Scott wrote:
> From: Jeremy M. Dolan <jmd@turbogeek.org> > >>Pluses: >> - clean up messy whitespace >> - cut precious picoseconds off compile time >> - cut kernel tree by 200k (+/- alot) >> >>Minuses: >> - adds 3.8M bzip2 or 4.7M gzip to next diff >> > > As someone who has spend a lot of time working on version control and > file merging, let be tell you the big minus you missed. > > After this patch go into the Linux kernel, everyone who is maintaining > a set of patches in parallel with the main kernel has a lot of extra > work resolving the conflicts caused by this change.
Which is why 2.5.1 is the only time it can go in. It would save 140K compressed (bz2) which kernel.org would probably appreciate, but it's probably too late already so the next best is always do it for subsequent patches. Perhaps their should be a patch_pp script that does runs both Lindent and strip_ws? Actually Alan Cox mentioned to me that pine had a bug where it stripped whitespace at end of lines :-) can't depend on that though.
Padraig.
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
|  |