From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?ISO-8859-1?Q?P=E1draig_Brady?= Subject: Re: [PATCH] Drop 80-character limit in checkpatch.pl Date: Fri, 18 Dec 2009 13:31:50 +0000 Message-ID: <4B2B8446.4060401@draigBrady.com> References: <200912171618.32882.bzolnier@gmail.com> <200912171708.07899.bzolnier@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Linus Torvalds Cc: Bartlomiej Zolnierkiewicz , =?ISO-8859-1?Q?Am=E9r?= =?ISO-8859-1?Q?ico_Wang?= , Mikulas Patocka , linux-kernel@vger.kernel.org, Alasdair G Kergon , dm-devel@redhat.com List-Id: dm-devel.ids On 17/12/09 16:21, Linus Torvalds wrote: > I'd be ok with changing the warning to 132 characters, which is anoth= er > perfectly fine historical limit. Or we can split the difference, and = say > "ok, 106 characters is too much". I don't care. But 80 characters is > causing too many idiotic changes. I find smaller limits help when you want to do side-by-side diffs, or `git blame` for example, but I agree there needs to be taste applied= , so as to not mangle the code. cheers, P=E1draig.