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.