Re: [PATCH] CodingStyle: Add information about trailing whitespace.

2007-06-28 Thread Andy Isaacson
On Wed, Jun 27, 2007 at 11:58:30PM -0700, Josh Triplett wrote: > Jan Engelhardt wrote: > > On Jun 28 2007 06:29, dave young wrote: > >> IMHO, another cause of trailing whitespace is human error, for > >> example long lines breaking will easy to cause the first line with one > >> traling whitespace

Re: [PATCH] CodingStyle: Add information about trailing whitespace.

2007-06-28 Thread Dave Young
7 2:59 PM > > To: Jan Engelhardt > > Cc: dave young; Chris Shoemaker; Josh Triplett; > linux-kernel@vger.kernel.org; > > [EMAIL PROTECTED] > > Subject: Re: [PATCH] CodingStyle: Add information about trailing > whitespace. > > > > Jan Engelhardt wrote: >

Re: [PATCH] CodingStyle: Add information about trailing whitespace.

2007-06-28 Thread Dave Young
h Triplett; linux-kernel@vger.kernel.org; > [EMAIL PROTECTED] > Subject: Re: [PATCH] CodingStyle: Add information about trailing whitespace. > > Jan Engelhardt wrote: > > On Jun 28 2007 06:29, dave young wrote: > >> IMHO, another cause of trailing whitespace is human error, for

RE: [PATCH] CodingStyle: Add information about trailing whitespace.

2007-06-28 Thread Li Yang-r58472
> Subject: Re: [PATCH] CodingStyle: Add information about trailing whitespace. > > Jan Engelhardt wrote: > > On Jun 28 2007 06:29, dave young wrote: > >> IMHO, another cause of trailing whitespace is human error, for > >> example long lines breaking will easy to

Re: [PATCH] CodingStyle: Add information about trailing whitespace.

2007-06-28 Thread Josh Triplett
Jan Engelhardt wrote: > On Jun 28 2007 06:29, dave young wrote: >> IMHO, another cause of trailing whitespace is human error, for >> example long lines breaking will easy to cause the first line with one >> traling whitespace (original space between the last two words). > > Most common errors (to

Re: [PATCH] CodingStyle: Add information about trailing whitespace.

2007-06-27 Thread Jan Engelhardt
On Jun 28 2007 06:29, dave young wrote: > > IMHO, another cause of trailing whitespace is human error, for > example long lines breaking will easy to cause the first line with one > traling whitespace (original space between the last two words). Most common errors (to me) are: - hit return+tab

Re: [PATCH] CodingStyle: Add information about trailing whitespace.

2007-06-27 Thread dave young
2007/6/28, Josh Triplett <[EMAIL PROTECTED]>: dave young wrote: > 2007/6/27, Jan Engelhardt <[EMAIL PROTECTED]>: >> 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 :) >> >>> Som

Re: [PATCH] CodingStyle: Add information about trailing whitespace.

2007-06-27 Thread Josh Triplett
dave young wrote: > 2007/6/27, Jan Engelhardt <[EMAIL PROTECTED]>: >> 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 somethin

Re: [PATCH] CodingStyle: Add information about trailing whitespace.

2007-06-27 Thread dave young
2007/6/27, Jan Engelhardt <[EMAIL PROTECTED]>: 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=flowe

Re: [PATCH] CodingStyle: Add information about trailing whitespace.

2007-06-27 Thread Jan Engelhardt
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

Re: [PATCH] CodingStyle: Add information about trailing whitespace.

2007-06-27 Thread Jan Engelhardt
On Jun 27 2007 12:31, Andrew Morton wrote: > >quilt has ways of detecting and/or correcting newly-added trailing >whitespace, but I don't know the details. Upon `quilt ref`, it usually barfs about every line you touched where you added or kept whitespace at EOL. >One could share the various scri

Re: [PATCH] CodingStyle: Add information about trailing whitespace.

2007-06-27 Thread Oleg Verych
> Newsgroups: gmane.linux.kernel > Date: Wed, 27 Jun 2007 12:31:22 -0700 > > > akpm:/usr/src/25> grep -l "^+.*[]$" patches/git-*.patch > patches/git-acpi.patch > patches/git-alsa.patch > patches/git-battery.patch > patches/git-cifs.patch > patches/git-drm.patch > patches/git-gfs2-nmw.patch

Re: [PATCH] CodingStyle: Add information about trailing whitespace.

2007-06-27 Thread Andrew Morton
On Wed, 27 Jun 2007 10:59:20 -0700 Josh Triplett <[EMAIL PROTECTED]> wrote: > +Do not leave trailing whitespace at the ends of lines. Some editors with > +"smart" indentation will insert whitespace at the beginning of new lines as > +appropriate, so you can start typing the next line of code righ

Re: [PATCH] CodingStyle: Add information about trailing whitespace.

2007-06-27 Thread Chris Shoemaker
On Wed, Jun 27, 2007 at 11:17:09AM -0700, Josh Triplett wrote: > On Wed, 2007-06-27 at 14:05 -0400, Chris Shoemaker wrote: > > On Wed, Jun 27, 2007 at 10:59:20AM -0700, Josh Triplett wrote: > > > +Do not leave trailing whitespace at the ends of lines. Some editors with > > > +"smart" indentation w

Re: [PATCH] CodingStyle: Add information about trailing whitespace.

2007-06-27 Thread Chris Shoemaker
On Wed, Jun 27, 2007 at 10:59:20AM -0700, Josh Triplett wrote: > +Do not leave trailing whitespace at the ends of lines. Some editors with > +"smart" indentation will insert whitespace at the beginning of new lines as > +appropriate, so you can start typing the next line of code right away. > +How

Re: [PATCH] CodingStyle: Add information about trailing whitespace.

2007-06-27 Thread Josh Triplett
On Wed, 2007-06-27 at 14:05 -0400, Chris Shoemaker wrote: > On Wed, Jun 27, 2007 at 10:59:20AM -0700, Josh Triplett wrote: > > +Do not leave trailing whitespace at the ends of lines. Some editors with > > +"smart" indentation will insert whitespace at the beginning of new lines as > > +appropriate

[PATCH] CodingStyle: Add information about trailing whitespace.

2007-06-27 Thread Josh Triplett
Signed-off-by: Josh Triplett <[EMAIL PROTECTED]> --- Documentation/CodingStyle | 12 1 files changed, 12 insertions(+), 0 deletions(-) diff --git a/Documentation/CodingStyle b/Documentation/CodingStyle index b49b92e..00bffa7 100644 --- a/Documentation/CodingStyle +++ b/Documentatio