Messages in this thread | | | Date | Thu, 28 Jun 2007 06:00:12 +0000 | From | "dave young" <> | Subject | Re: [PATCH] CodingStyle: Add information about trailing whitespace. |
| |
2007/6/27, Jan Engelhardt <jengelh@computergmbh.de>: > > On Jun 27 2007 14:05, Chris Shoemaker wrote: > > > >What I'd really like to see is, _why_ is trailing whitespace > >considered harmful? > > Consumes bytes you'll never see :) > > >Something about MUAs not preserving it or something? > > Well, there is format=flowed. text/plain mails with a trailing blank at the end > of line indicates the paragraph continues -- as a curtesy for mail readers with > non-fixed-font to display the paragraph coherently rather than breaking it at > 72/80 columns. > > > Jan > -- > - > 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/ >
There's more potential to cause lines end with whitespace. How about remove the "Some editors with ..." like this :
+Do not leave trailing whitespace at the ends of lines. + +Git will warn you about patches that introduce trailing whitespace, and can +optionally strip the trailing whitespace for you; however, if applying a series +of patches, this may make later patches in the series fail by changing their +context lines. + - 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/
| |