From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from out3-smtp.messagingengine.com (out3-smtp.messagingengine.com [66.111.4.27]) by dpdk.org (Postfix) with ESMTP id DDDA3160 for ; Mon, 29 Oct 2018 03:19:00 +0100 (CET) Received: from compute1.internal (compute1.nyi.internal [10.202.2.41]) by mailout.nyi.internal (Postfix) with ESMTP id 6DFA721FC0; Sun, 28 Oct 2018 22:19:00 -0400 (EDT) Received: from mailfrontend1 ([10.202.2.162]) by compute1.internal (MEProxy); Sun, 28 Oct 2018 22:19:00 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=monjalon.net; h= from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding:content-type; s=mesmtp; bh=Z+JjJ4XwGlTiZ+JoTit4GNr2SpzUlrKMwm97odPnr1Q=; b=JNyMMTu6+w8Y /9DwzVEYcx8NKctPNWwcyAK/u8YDLF5osmqRVGDX//UHp4mLzn2P/WU/28jAlHF6 mWDtpm17XLAUAr9OvCx3J6tjdg+iNSNhatC32Nps8QbFCKXAWQL+fNkIsnizpmeq khYtcHsIYP+KOmKsVVR6UKYkZRWiU7w= 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-proxy:x-me-proxy:x-me-sender:x-me-sender :x-sasl-enc; s=fm1; bh=Z+JjJ4XwGlTiZ+JoTit4GNr2SpzUlrKMwm97odPnr 1Q=; b=InV6ALY+I5EHfAodJFSPBm2zW3gzAIWmeaWnSFpKovJgQpZQQCrZzc+U7 QB7dQtuF+QXr+XGdwc+WJqIBv64MnFpy6K5ke6iPjJZccEyL4czwVSMaO9X+iRyH ZndEptLGxyRdHt+dQX+GRHfDdC9AExWkPLJuoD2IloeNQRxZ+jueFUVyYlPB7NNs iAQIYzZH91ScfPPmwIFn0PzbO+T+zUy/xxDfa4VUF3uxXLbCNMpK+cxBq5/J4GfU RZ/vRNJWrKkO087I46FQNR2ue/tcKGV5SiAK72mPoFI4pq2Lyz0rb45mZxCInz9D Pp/YIdEYgS4sZtCyzsfZNh28H/yDA== X-ME-Sender: X-ME-Proxy: Received: from xps.localnet (184.203.134.77.rev.sfr.net [77.134.203.184]) by mail.messagingengine.com (Postfix) with ESMTPA id F19CBE43AC; Sun, 28 Oct 2018 22:18:58 -0400 (EDT) From: Thomas Monjalon To: "Joseph, Anoob" Cc: dev@dpdk.org, Ferruh Yigit , Bruce Richardson , Pablo de Lara , Jerin Jacob , Narayana Prasad Date: Mon, 29 Oct 2018 03:19:05 +0100 Message-ID: <2612293.rcIgIV396C@xps> In-Reply-To: <789e1a1a-aa7b-8aa0-41b9-ce188eeccc6b@intel.com> References: <1538995285-2040-1-git-send-email-anoob.joseph@caviumnetworks.com> <98620a3a-2968-be0b-4690-ee4d6dad1101@caviumnetworks.com> <789e1a1a-aa7b-8aa0-41b9-ce188eeccc6b@intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Subject: Re: [dpdk-dev] [PATCH 0/8] examples/l2fwd: fix checkpatch reported issues 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: Mon, 29 Oct 2018 02:19:01 -0000 08/10/2018 14:35, Ferruh Yigit: > On 10/8/2018 12:29 PM, Joseph, Anoob wrote: > > On 08-10-2018 16:30, Ferruh Yigit wrote: > >> On 10/8/2018 11:41 AM, Anoob Joseph wrote: > >>> This patchset fixes multiple issues reported by checkpatch in l2fwd > >>> code base. These issues would be flagged for any new copy of the file > >>> and hence, fixing at the source. > >>> > >>> Anoob Joseph (8): > >>> examples/l2fwd: remove quoted white space before newline > >>> examples/l2fwd: need space between two args > >>> examples/l2fwd: else should follow close brace > >>> examples/l2fwd: replace bare usage of 'unsigned' > >>> examples/l2fwd: follow convention for block comments > >>> examples/l2fwd: limit line to 80 char > >>> examples/l2fwd: space required between elements > >>> examples/l2fwd: remove null initialisation > >> Hi Anoob, > >> > >> I am not sure if it is good idea to get syntax only fixes, I would prefer to get > >> syntax fixes when some other code touches that area. > > I'm preparing a new copy of l2fwd with support for eventmode(as > > suggested in, > > https://mails.dpdk.org/archives/dev/2018-August/109717.html). All these > > issues were flagged when I did the copy. What would be the better > > approach in that case? Fix it in just the new application or fix the > > source? Fixing in just the new app would mean, the two versions will > > have a diff at the time of copy. > > I see, make sense to not create syntax diff copied and original versions, I > missed the new copy part. So OK for this patch. It was said that there will be not so much common code. So why bothering to reformat the original example? Anyway, if such cleanup is worth before duplicating, please insert it in the same patchset as the new example.