... :49:01 2008 +0200 >> >> checkpatch: relax spacing and line length >> >> We all had ... with > either by rejecting your patch or fixing it themselves; it is ... an array/struct >> + # definition/initializer. Spaces ok -- used for align >> + # on ...

linux-kernel - Benny Halevy - Apr 9 2008 - 08:10

... 49:01 2008 +0200 > > checkpatch: relax spacing and line length > > We all ... with either by rejecting your patch or fixing it themselves; it ... "\n"; > + ERROR("use tabs not spaces\n" . $herevet); > + } > } -apw -- To unsubscribe ...

linux-kernel - Andy Whitcroft - Apr 8 2008 - 13:12

... 0 errors, 0 warnings, 36 lines checked d27a9f5.diff has no ... 01 2008 +0200 checkpatch: relax spacing and line length We all ... Engelhardt --- scripts/checkpatch.pl | 22 ++++++++++++++++------ 1 files changed ...

linux-kernel - Jan Engelhardt - Apr 6 2008 - 00:54

... d27a9f5.diff has no obvious style problems and is ready for submission. > === > commit ... :49:01 2008 +0200 > > checkpatch: relax spacing and line length > > We all had ... . It's a tool for patch submitters and maintainers that automates a part ...

linux-kernel - Adrian Bunk - Apr 6 2008 - 06:08

... org> wrote: > >>> To justify changing checkpatch to loosen its checks I ... using tabs for indent and then spaces for alignment > is functionally ... now* therefore we need to relax it, not keep it the ... we add more details to relax the requirements but > overall we ...

linux-kernel - Andy Whitcroft - Apr 9 2008 - 16:16

... in Kconfig, spaces are never used for indentation" Although CodingStyle and checkpatch just ... the current indentation style, just to relax the rules to allow a superset of ... program so I can easily quote patches or code snippets.] Benny -- To ...

linux-kernel - Benny Halevy - Apr 15 2008 - 05:09

... This will reduce the usefulness of checkpatch for those developers who > choose to ... -column limit. > >> This also relaxes the tab doctrine, because spaces DO make sense ... unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the ...

linux-kernel - Benny Halevy - Apr 6 2008 - 07:52

... .org> wrote: >>> To justify changing checkpatch to loosen its checks I ... using tabs for indent and then spaces for alignment is functionally ... *now* therefore we need to relax it, not keep it the ... we add more details to relax the requirements but overall we ...

linux-kernel - Benny Halevy - Apr 9 2008 - 13:02

... will reduce the usefulness of checkpatch for those developers who choose ... limit. > This also relaxes the tab doctrine, because spaces DO make ... a mix of tabs and non-tabs and looks a mess ... from this list: send the line "unsubscribe linux-kernel" in the ...

linux-kernel - Andrew Morton - Apr 6 2008 - 01:18

... ends as it loads the patches up. I thought that this ... on the context including those spaces. Surely, this would lead to ... patches at once", 2) accepting checkpatch only changes is stupid, and ... from this list: send the line "unsubscribe linux-kernel" in the ...

linux-kernel - Andy Whitcroft - Apr 9 2008 - 09:14

... This also relaxes the tab doctrine, because spaces DO make sense ... kernel all tabs are 8 spaces wide. > > When you view the ... setting that's your fault. And for other kernel developers tabs ... from this list: send the line "unsubscribe linux-kernel" in the ...

linux-kernel - Sam Ravnborg - Apr 6 2008 - 07:08

... /tools issue more than a checkpatch one? > > I think you should ... support the tabs to level, spaces to align style. > > g'luck ... s output because checkpatch.pl forbids it, and I get scorned ... from this list: send the line "unsubscribe linux-kernel" in the ...

linux-kernel - Boaz Harrosh - Apr 7 2008 - 05:51

... > wrote: >>> This also relaxes the tab doctrine, because spaces DO make sense ... to the program's structure: syntax and coding style. Benny > > Sam > -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body ...

linux-kernel - Benny Halevy - Apr 7 2008 - 12:37

... number spaces to match up logically with the >previous line is ... Kernel Emacs >cc-mode style and cc-mode works like the ... recipient of format changes only patch bombs >before. I suppose you ... oh-I-decided-to-run-checkpatch-on-existing-code bombs. -- To ...

linux-kernel - Jan Engelhardt - Apr 11 2008 - 00:24

... apw@shadowen.org> writes: > > > > > > To justify changing checkpatch to loosen its checks I would hope ... > Here was a concrete proposal (with patch for CodingStyle) from my side some ... favour, and > slightly happier to have the string on the printk line for ...

linux-kernel - Andy Whitcroft - Apr 9 2008 - 04:30

... that idiom I fall over stunned and > > can't get up again. > ... pushed out a testing version of checkpatch which is identicle the 0.18 ... likely be dipping into email now and again. -apw -- To unsubscribe from this list: send the line "unsubscribe ...

linux-kernel - Andy Whitcroft - Apr 11 2008 - 11:54

... an editor/tools issue more than a checkpatch one? I think you should first try ... etc) to support the tabs to level, spaces to align style. g'luck with that. ... To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of ...

linux-kernel - Joe Perches - Apr 6 2008 - 14:52

... writes: > > > > To justify changing checkpatch to loosen its checks I would hope ... > Here was a concrete proposal (with patch for CodingStyle) from my side some > ... favour, and slightly happier to have the string on the printk line for ...

linux-kernel - Andy Whitcroft - Apr 9 2008 - 04:19

... 'm not sure what "two space change" proposal this Steve referred ... working editing habits *or* to patch That's working well for ... in the CodingStyle document and in checkpatch.pl in addition to ... from this list: send the line "unsubscribe linux-kernel" in the ...

linux-kernel - Benny Halevy - Apr 13 2008 - 05:53

... writes: > > To justify changing checkpatch to loosen its checks I would ... Here was a concrete proposal (with patch for CodingStyle) from my side some ... unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body ...

linux-kernel - Andi Kleen - Apr 8 2008 - 14:01