[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <Y8t8SZ9VGm5fjYAn@sol.localdomain>
Date: Fri, 20 Jan 2023 21:46:49 -0800
From: Eric Biggers <ebiggers@...nel.org>
To: Yangtao Li <frank.li@...o.com>
Cc: jaegeuk@...nel.org, chao@...nel.org, linux-kernel@...r.kernel.org,
linux-f2fs-devel@...ts.sourceforge.net
Subject: Re: [f2fs-dev] [PATCH v2 1/5] f2fs: fix unnecessary newlines in
super.c
On Sat, Jan 21, 2023 at 01:12:00PM +0800, Yangtao Li via Linux-f2fs-devel wrote:
> commit bdc48fa11e46 ("checkpatch/coding-style: deprecate 80-column
> warning") increased the limit to 100 columns. Let's remove some
> unnecessary newlines.
80 is still the *preferred* limit. See Documentation/process/coding-style.rst:
The preferred limit on the length of a single line is 80 columns.
Statements longer than 80 columns should be broken into sensible chunks,
unless exceeding 80 columns significantly increases readability and does
not hide information.
Also, to avoid churn, patches that just change whitespace generally aren't
accepted, unless they are changing the relevant code anyway.
- Eric
Powered by blists - more mailing lists