From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 56ED3A0517; Wed, 10 Jun 2020 07:22:57 +0200 (CEST) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 39EA12B83; Wed, 10 Jun 2020 07:22:57 +0200 (CEST) Received: from mail-io1-f65.google.com (mail-io1-f65.google.com [209.85.166.65]) by dpdk.org (Postfix) with ESMTP id 2B2F81DB8 for ; Wed, 10 Jun 2020 07:22:56 +0200 (CEST) Received: by mail-io1-f65.google.com with SMTP id c8so777638iob.6 for ; Tue, 09 Jun 2020 22:22:56 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=Yzd3gYiWiiVNKXMimb7yanVrp6POK8rkof5tzTKC4tw=; b=P0uo2JOrSSjuZ3JGpilNfFlcVVeuZFcYTGRBFeRY9E/oT1MbjShYhyo6A2Yi+/lqhW yMSQi8QDTCS7qfIhNaff+oLL0TIKmDUSvZARjx7Ttk8Jvu2InVC2Bp4G9wQqZZOfskiV 9yI3ffrCcBY2G1QTzx3Q1dtQgquxIFcf1nXeR5E4CBfqtxbvI91CAKDDPI5HkgzFtB4W NnCHMNlbrcYa37RakoTG4oVo7cfSl8ut2WejJ2MENhlueML1YW5zXjpg6jvRqhdiFVL8 RSp4Nvd52Xc90mU/H6suyY7UrgdXRa+33Arb58EaCrdXDGx3POX2z5NKJk+cREKJkWMq x1XQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=Yzd3gYiWiiVNKXMimb7yanVrp6POK8rkof5tzTKC4tw=; b=BQQt7GOBVIzQacdvAnKBM4dtXEkWy5qdXNWGJLlDHojbmbZHG4g4PbFSbD78prcxDS Eefd9HWu9cGvDL783QluvWtO5s47hw92JzcqVjuA0y6yU5oSvtesxRF2l7VyE2VyDe6h 7pJHqHFiAIGjfKFyyy9CkbFMOkCUmff5CiAXKAaLWDsFEGepoCOPHPkg6EVBSMWrympu f2SKkjs2UwOf+Ydl0HlE7+GAKgcBoCGLn0ek0XvQyUtIn7NkJfcGq9K7IJjaSym+0AHo q2Bu/cdrrSh8qCMQ+ga5EnXQ9GqpWpKY0ygQXSuMk8MOhmK0x66y+mhtkQ4bNPqLH119 lMmg== X-Gm-Message-State: AOAM532Vnn0wfJpcbT51+iDFngR/pPTIQvsShCqnlZcUyfn+1fkvtdhQ L6fIgV3tSiOXJGMfFvBUipo0DxhpM2aNNdoRj78= X-Google-Smtp-Source: ABdhPJyds0MKHQAiI2KOlzzynh4ZeIYDE4AfWx3JWe8KQcXMQCSX25GkftHOQ/6kk+ZKU+JLSuRxlNeks3CiY9DrXXw= X-Received: by 2002:a5d:8cc1:: with SMTP id k1mr1699290iot.123.1591766575369; Tue, 09 Jun 2020 22:22:55 -0700 (PDT) MIME-Version: 1.0 References: <20200608164640.189755-1-bruce.richardson@intel.com> <20200608121723.7a042105@hermes.lan> <20200609092949.GA1565@bricha3-MOBL.ger.corp.intel.com> <5710188e-4bba-c909-34cb-1bb67cbc3af0@solarflare.com> <20200609135717.GA1583@bricha3-MOBL.ger.corp.intel.com> In-Reply-To: <20200609135717.GA1583@bricha3-MOBL.ger.corp.intel.com> From: Jerin Jacob Date: Wed, 10 Jun 2020 10:52:39 +0530 Message-ID: To: Bruce Richardson Cc: Andrew Rybchenko , "Ananyev, Konstantin" , Stephen Hemminger , "dev@dpdk.org" Content-Type: text/plain; charset="UTF-8" Subject: Re: [dpdk-dev] [PATCH RFC] devtools: increase default line length to 100 X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On Tue, Jun 9, 2020 at 7:27 PM Bruce Richardson wrote: > > On Tue, Jun 09, 2020 at 04:40:28PM +0300, Andrew Rybchenko wrote: > > On 6/9/20 1:00 PM, Ananyev, Konstantin wrote: > > > > > >> > > >> On Mon, Jun 08, 2020 at 12:17:23PM -0700, Stephen Hemminger wrote: > > >>> On Mon, 8 Jun 2020 17:46:40 +0100 Bruce Richardson > > >>> wrote: > > >>> > > >>>> Rather than continuing to recommend an 80-char line limit, let's > > >>>> take a hint from the Linux kernel[1] and aim for an 100-char > > >>>> recommended limit instead. > > >>>> > > >>>> [1] > > >>>> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=bdc48fa11e46f867ea4d75fa59ee87a7f48be144 > > >>>> > > >>>> Signed-off-by: Bruce Richardson --- > > >>>> devtools/checkpatches.sh | 2 +- > > >>>> doc/guides/contributing/coding_style.rst | 2 +- 2 files changed, 2 > > >>>> insertions(+), 2 deletions(-) > > >>>> > > >>>> diff --git a/devtools/checkpatches.sh b/devtools/checkpatches.sh > > >>>> index 158087f1c..4970ed830 100755 --- a/devtools/checkpatches.sh +++ > > >>>> b/devtools/checkpatches.sh @@ -15,7 +15,7 @@ > > >>>> VALIDATE_NEW_API=$(dirname $(readlink -f $0))/check-symbol-change.sh > > >>>> # Codespell can also be enabled by setting DPDK_CHECKPATCH_CODESPELL > > >>>> to a valid path # to a dictionary.txt file if dictionary.txt is not > > >>>> in the default location. > > >>>> codespell=${DPDK_CHECKPATCH_CODESPELL:-enable} > > >>>> -length=${DPDK_CHECKPATCH_LINE_LENGTH:-80} > > >>>> +length=${DPDK_CHECKPATCH_LINE_LENGTH:-100} > > >>>> > > >>>> # override default Linux options options="--no-tree" diff --git > > >>>> a/doc/guides/contributing/coding_style.rst > > >>>> b/doc/guides/contributing/coding_style.rst index > > >>>> 4efde93f6..1db3a7bbe 100644 --- > > >>>> a/doc/guides/contributing/coding_style.rst +++ > > >>>> b/doc/guides/contributing/coding_style.rst @@ -21,7 +21,7 @@ The > > >>>> rules and guidelines given in this document cannot cover every > > >>>> situation, so * In the case of creating new files, the style should > > >>>> be consistent within each file in a given directory or module. * > > >>>> The primary reason for coding standards is to increase code > > >>>> readability and comprehensibility, therefore always use whatever > > >>>> option > > >> will make the code easiest to read. > > >>>> > > >>>> -Line length is recommended to be not more than 80 characters, > > >>>> including comments. +Line length is recommended to be not more than > > >>>> 100 characters, including comments. [Tab stop size should be > > >>>> assumed to be 8-characters wide]. > > >>>> > > >>>> .. note:: > > >>> > > >>> I would even support going to 120 characters. > > >>> > > >>> Acked-by: Stephen Hemminger > > >> > > >> I think 100 is enough. > > >> > > >> In my case, I have a 1080p 24" monitor, and with two terminals > > >> side-by-side 100 characters just fits inside each vim window. Going to > > >> 120 would be fine for single terminal at a time, but I would find > > >> awkward for e.g. side-by-side diff comparison in meld etc. > > > > > > My preference would be to keep things as it is - 80 chars per line. > > > Having multiple different formatting styles in one source file looks > > > really awkward and make it hard to follow. > > > > +1 > > > I wouldn't personally consider increasing the max line length as a style > change, but even if you consider it such I'd worry about rejecting style > changes on the basis that it may be different to what is there before. That > logic means that we can never, ever change any element of DPDK coding style. > > I can see the issue with changes that require us to rework the style of > code in order to comply with the new style, but changing the max length > from 80 to 100 does not make 80-char lines incorrect and needing changes. Another point is: Other projects derived from the Linux kernel coding standard also getting migrated to the new coding standard. This change would be useful for: a) People works on multiple Linux coding standard derived projects b) Some of the code such as 'base' and 'common' code for HW drivers are shared between multiple projects. Such code needs adjustment/change when pulling to the DPDK code base it it still follows 80 chars per line. > > Regards, > /Bruce