From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from out2-smtp.messagingengine.com (out2-smtp.messagingengine.com [66.111.4.26]) by dpdk.org (Postfix) with ESMTP id E3D3D1CD31 for ; Sun, 13 May 2018 15:58:55 +0200 (CEST) Received: from compute1.internal (compute1.nyi.internal [10.202.2.41]) by mailout.nyi.internal (Postfix) with ESMTP id 305BF25D5A; Sun, 13 May 2018 09:58:55 -0400 (EDT) Received: from mailfrontend1 ([10.202.2.162]) by compute1.internal (MEProxy); Sun, 13 May 2018 09:58:55 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=monjalon.net; h= cc:content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to:x-me-sender :x-me-sender:x-sasl-enc; s=mesmtp; bh=/x+Dess6H/i18QvWzn8B7v7feM Vq/UGq1RyCCqMLOOU=; b=hX1Yxbdk1zUoVx731X70pNW6SMG5tqY1Za8j8OUAD+ 6EttCAh0GfCoxJe6UTU2TsnGM05D7fDs/WUQroc/8C8qRV8v7cZ+EBg6qOfqHYGQ xul0ayuiAXp+Ad5irrRBz2IkLWPJCSQwTCVKnGRXVPapoxK8rgPyLh9s+OQjZ8Z6 A= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:content-type :date:from:in-reply-to:message-id:mime-version:references :subject:to:x-me-sender:x-me-sender:x-sasl-enc; s=fm2; bh=/x+Des s6H/i18QvWzn8B7v7feMVq/UGq1RyCCqMLOOU=; b=G5/sTcoCpwkkIHijdJJm/d NlYhXE8SKKAkJ5B6T3tzb/qKvJBLj+chbDbeb7hGI1X6jpVHVVcccd4Up5cym3nT Uhg97n7tHSA7lnYdnJB6RwwQXCbZSvEvM5XmRPj8mSUcnOz0Wz+rCfPXOc4F4s7s F5f9+ZW0bEe2IyIxMkvDgQzpsdSnqkBiZE5JMIcRH1agFW1DatOuv78aE81msmN8 4OJvhp2QFC4UiFJcFMA4NUWYCPUbRXJSX79TrJcHZfwj+nUxT7fg7jspH4xBSrC6 sBFx6845BshzLQkkkyCoLBhsUZUhKNHwcTHH6edCHgAinNf/q/vMyhdykWie0GPQ == X-ME-Sender: Received: from xps.localnet (184.203.134.77.rev.sfr.net [77.134.203.184]) by mail.messagingengine.com (Postfix) with ESMTPA id 6437DE4919; Sun, 13 May 2018 09:58:54 -0400 (EDT) From: Thomas Monjalon To: Andy Green Cc: dev@dpdk.org, Stephen Hemminger , Jerin Jacob Date: Sun, 13 May 2018 15:58:53 +0200 Message-ID: <1962043.pv7O6beyaF@xps> In-Reply-To: References: <152591991920.119328.14523975619615362920.stgit@localhost.localdomain> <20180510080142.0c49116f@xeon-e3> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Subject: Re: [dpdk-dev] [PATCH v3 00/40] Fix build on gcc8 and various bugs 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: , X-List-Received-Date: Sun, 13 May 2018 13:58:56 -0000 Hi, 11/05/2018 02:29, Andy Green: > > On 05/10/2018 11:01 PM, Stephen Hemminger wrote: > > On Thu, 10 May 2018 20:13:31 +0800 > > Andy Green wrote: > > > >> I appreciate the reply. > >> > >> But why bother having a subject line at all if it is going to be > >> mechanically enforced that nothing in it is allowed to be "useful"? > >> That really doesn't make sense does it. > > > > It was done because there were lots of clueless patches showing > > up on the driver development list which had useless subject > > lines. Yes, the title should help to quickly identify the scope of the commits. Usually, giving variable names, function names, etc are not very useful. It is very common that some developers say "fixing function X" instead of describing the fixed behaviour: "fixing feature Y". > The "cure" is worse than the disease... It is a tool showing some warnings. We must be smart when using this tool (as any other tools) and consider that some warnings are false positive. > - I can mention, eg, that something changed to an int. But a size_t > or my_type_t? I am not allowed to mention it even if that is the whole > reason for the patch. Of course you can use size_t in the title if it is relevant. Sometimes, changing a type is fixing a feature, so better to name the feature. > - I can mention most libc apis, but not those that happen to have an > underscore, eg, timerfd_create(), even if that was the focus of the patch. > > - Any kind of manifest constant like MY_CONSTANT: illegal to mention, > even if the patch's job is change MY_CONSTANT to, say, 5. What should I > entitle that patch? "lib: change something to 5"? "lib: change > MY.CONSTANT to 5"? > > - I can mention most filenames or paths, eg, down /proc, or myfile.c. > But not if the filepath happens to contain an underscore. Even if the > effect of the patch is to migrate stuff from myfile.c to my_files/ > > The results are arbitrary... please consider removing this now it has > been in place a while and made its original point. I hope you got my point that the title should be a high level description of the scope or goal of the patch. The details are inside the commit log. But using underscore is accepted in the title if relevant. Every strong rules are stupids, that's why we have only guidelines and we are flexible.