Re: [perl #40543] [NEW] Test for space after curly braces

2006-10-18 Thread Paul Cochrane
this is too noisy as ~700 tests. please convert it to one test, like the other coding standard tests. Ok. Have changed it to be the same as the other coding standard tests. See attached patch. also, it should apply to perl files, as well. in fact, it should probably apply to most files in MAN

Re: [perl #40543] [NEW] Test for space after curly braces

2006-10-17 Thread Will Coleda
On Oct 17, 2006, at 4:25 PM, Jerry Gay via RT wrote: On Tue Oct 17 07:33:02 2006, [EMAIL PROTECTED] wrote: Well, the verdict defnitely seems to be that trailing space and tab characters are annoyances that should go away :-) This patch adds a new test (in place of the curly-space test I poste

[perl #40543] [NEW] Test for space after curly braces

2006-10-17 Thread Jerry Gay via RT
On Tue Oct 17 07:33:02 2006, [EMAIL PROTECTED] wrote: > Well, the verdict defnitely seems to be that trailing space and tab > characters are annoyances that should go away :-) This patch adds a > new test (in place of the curly-space test I posted earlier) which > searches for superfluous trailing

Re: [perl #40543] [NEW] Test for space after curly braces

2006-10-17 Thread Paul Cochrane
Well, the verdict defnitely seems to be that trailing space and tab characters are annoyances that should go away :-) This patch adds a new test (in place of the curly-space test I posted earlier) which searches for superfluous trailing spaces in source files. Comments definitely welcome! Regar

Re: [perl #40543] [NEW] Test for space after curly braces

2006-10-16 Thread Mark J. Reed
On 10/16/06, jerry gay <[EMAIL PROTECTED]> wrote: for vim users, these settings may help find trailing spaces and leading tabs: set list set listchars=trail:-,tab:\.\ note in this case, the trailing space on the second line is intentional. ~jerry Note that ":set list" all by itself will

Re: [perl #40543] [NEW] Test for space after curly braces

2006-10-16 Thread Leopold Toetsch
Am Montag, 16. Oktober 2006 19:05 schrieb jerry gay: > i see no reason to keep this specific to curlies. > i hate all useless trailing whitespace equally :) s/useless// Me too, but there are still some test files with ws at the end, e.g. t/op/sprintf_tests You can stare at test result compares i

Re: [perl #40543] [NEW] Test for space after curly braces

2006-10-16 Thread jerry gay
On 10/16/06, jerry gay <[EMAIL PROTECTED]> wrote: i see no reason to keep this specific to curlies. i hate all useless trailing whitespace equally :) for vim users, these settings may help find trailing spaces and leading tabs: set list set listchars=trail:-,tab:\.\ note in this case, the t

Re: [perl #40543] [NEW] Test for space after curly braces

2006-10-16 Thread Mark J. Reed
On 10/16/06, chromatic <[EMAIL PROTECTED]> wrote: On Monday 16 October 2006 02:00, Paul Cochrane wrote: > This patch adds a test to the coding standards which checks for space > or tab characters after a curly brace when the curly is the last > printable character on the line. The pumpking hasn

Re: [perl #40543] [NEW] Test for space after curly braces

2006-10-16 Thread jerry gay
On 10/16/06, chromatic <[EMAIL PROTECTED]> wrote: On Monday 16 October 2006 02:00, Paul Cochrane wrote: > This patch adds a test to the coding standards which checks for space > or tab characters after a curly brace when the curly is the last > printable character on the line. The pumpking hasn

Re: [perl #40543] [NEW] Test for space after curly braces

2006-10-16 Thread chromatic
On Monday 16 October 2006 02:00, Paul Cochrane wrote: > This patch adds a test to the coding standards which checks for space > or tab characters after a curly brace when the curly is the last > printable character on the line. The pumpking hasn't explicitly > declared that this should be a codin

[perl #40543] [NEW] Test for space after curly braces

2006-10-16 Thread Paul Cochrane
# New Ticket Created by "Paul Cochrane" # Please include the string: [perl #40543] # in the subject line of all future correspondence about this issue. # http://rt.perl.org/rt3/Ticket/Display.html?id=40543 > Hi, This patch adds a test to the coding standards which checks for space or tab cha