checkpatch.pl: remove the 80 charactes punch card limit
in [Kernel]
|
Prev: Disable SR when more than one pipe is enabled
Next: [PATCH 3/3] [RFC] nmi_watchdog: config option to enable new nmi_watchdog
From: Andi Kleen on 28 Jan 2010 11:00 Stefani Seibold <stefani(a)seibold.net> writes: > The time of 80 characters punch card and terminals are over, so i would > be a good thing to set the line length limit to 120. Every display today > should be able handle this. > > Signed-off-by: Stefani Seibold <stefani(a)seibold.net> Full Ack! -Andi -- ak(a)linux.intel.com -- Speaking for myself only. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
From: Krzysztof Halasa on 28 Jan 2010 11:10 Stefani Seibold <stefani(a)seibold.net> writes: > The time of 80 characters punch card and terminals are over, so i would > be a good thing to set the line length limit to 120. Every display today > should be able handle this. I thought it's been already agreed that the limit is lifted altogether? Not even the 132 that was considered by some people sane enough. The other thing is that fine "code complexity", but I think checkpatch can't check it ATM, can it? -- Krzysztof Halasa -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
From: David Daney on 28 Jan 2010 13:00 Andi Kleen wrote: > Stefani Seibold <stefani(a)seibold.net> writes: > >> The time of 80 characters punch card and terminals are over, so i would >> be a good thing to set the line length limit to 120. Every display today >> should be able handle this. >> >> Signed-off-by: Stefani Seibold <stefani(a)seibold.net> > > Full Ack! > > -Andi I agree, but if people still have problems with it, perhaps reevaluating Joe Perches' patch: http://lkml.org/lkml/2009/12/18/3 could be an alternative. David Daney -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
From: Alexander Clouter on 28 Jan 2010 13:10 Andi Kleen <andi(a)firstfloor.org> wrote: > >> The time of 80 characters punch card and terminals are over, so i would >> be a good thing to set the line length limit to 120. Every display today >> should be able handle this. >> >> Signed-off-by: Stefani Seibold <stefani(a)seibold.net> > > Full Ack! > NACK Attack! There is a reason Knuth only put so many words on a line with TeX. Eugh. Cheers -- Alexander Clouter ..sigmonster says: Keep out of reach of children. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
From: tytso on 29 Jan 2010 13:40
On Thu, Jan 28, 2010 at 04:39:29PM +0000, Alexander Clouter wrote: > Andi Kleen <andi(a)firstfloor.org> wrote: > > > >> The time of 80 characters punch card and terminals are over, so i would > >> be a good thing to set the line length limit to 120. Every display today > >> should be able handle this. > >> > >> Signed-off-by: Stefani Seibold <stefani(a)seibold.net> > > > > Full Ack! > > > NACK Attack! > > There is a reason Knuth only put so many words on a line with TeX. Linus has already agreed that the 80 character limit is stupid. Let's just nuke this check from checkpatch.pl and move on. - Ted -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo(a)vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/ |