From patchwork Thu Nov 19 11:58:50 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ferruh Yigit X-Patchwork-Id: 84370 Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 6B2D5A04DD; Thu, 19 Nov 2020 12:59:13 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 54934F90; Thu, 19 Nov 2020 12:59:11 +0100 (CET) Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id 0B36523D for ; Thu, 19 Nov 2020 12:59:08 +0100 (CET) IronPort-SDR: wgB0SUPiZGsiotvnagVoy4gnWbXZrpW+oZmLPOcev+LG1rSiUpn6qhhBkoK9ywATPz0REtCTk3 3BW9yFfdR74A== X-IronPort-AV: E=McAfee;i="6000,8403,9809"; a="170494376" X-IronPort-AV: E=Sophos;i="5.77,490,1596524400"; d="scan'208";a="170494376" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga002.jf.intel.com ([10.7.209.21]) by fmsmga103.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 19 Nov 2020 03:59:06 -0800 IronPort-SDR: CWG5wtF+QPKwl5+EH5BpjBq05uxD6JhXrv2SAErY6UURci8AVF1Xu+L+NAummOeIo8icnbfrIB /xWm7AC73idw== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.77,490,1596524400"; d="scan'208";a="341665093" Received: from silpixa00399752.ir.intel.com (HELO silpixa00399752.ger.corp.intel.com) ([10.237.222.180]) by orsmga002.jf.intel.com with ESMTP; 19 Nov 2020 03:59:05 -0800 From: Ferruh Yigit To: dev@dpdk.org Cc: Ferruh Yigit Date: Thu, 19 Nov 2020 11:58:50 +0000 Message-Id: <20201119115900.4095566-1-ferruh.yigit@intel.com> X-Mailer: git-send-email 2.26.2 In-Reply-To: <20200824094021.2323605-1-ferruh.yigit@intel.com> References: <20200824094021.2323605-1-ferruh.yigit@intel.com> MIME-Version: 1.0 Subject: [dpdk-dev] [v21.02 v3 00/10] cppcheck 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" Fix a few of issues identified by cppcheck. There are more to address, cppcheck can be run as: "cppcheck --enable=all --force ." These are very trivial issues, I think can be postponed to next release. Ferruh Yigit (10): app/procinfo: fix redundant condition app/procinfo: fix negative check on unsigned variable app/procinfo: remove suspicious sizeof app/procinfo: remove useless assignment net/pcap: remove local variable shadowing outer one net/bonding: remove local variable shadowing outer one net/af_xdp: remove useless assignment net/bnxt: fix redundant return app/crypto-perf: remove always true condition net/avp: remove always true condition app/proc-info/main.c | 16 +++++----------- app/test-crypto-perf/main.c | 4 ++-- drivers/net/af_xdp/rte_eth_af_xdp.c | 1 - drivers/net/avp/avp_ethdev.c | 2 +- drivers/net/bnxt/tf_core/tf_em_common.c | 1 - drivers/net/bonding/rte_eth_bond_8023ad.c | 6 ++---- drivers/net/pcap/rte_eth_pcap.c | 5 ++--- 7 files changed, 12 insertions(+), 23 deletions(-)