Message ID | 20220202094802.3618978-23-sean.morrissey@intel.com (mailing list archive) |
---|---|
State | Superseded, archived |
Delegated to: | Thomas Monjalon |
Headers |
Return-Path: <dev-bounces@dpdk.org> X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 3E555A00C5; Wed, 2 Feb 2022 10:50:28 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 7EA6F4272D; Wed, 2 Feb 2022 10:49:07 +0100 (CET) Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by mails.dpdk.org (Postfix) with ESMTP id 26576411FE for <dev@dpdk.org>; Wed, 2 Feb 2022 10:49:04 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1643795345; x=1675331345; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=550ljhVlbQblPCB5CYHexqerWPnSDdajSYhG1wHQrxo=; b=eCO//IJY1yFnRIzFbwhEtFJ+v4Yzcl5p5KNq+RLs8M/ZufHqZEJQbIOh jDA3diKDF2v+H3Q3Fh2bOuilsZbRQFZ0ce/MbPrN7tz3ralhuBblKiC7Q vs30iEIgHB4dplKRMEqqhGn7DNq+vmE9UeFs1x5hYdWvGbNK1Ht3LdPVx Juts4uqWKjBytAvA1Z8Ilv/iY6Za05iUHGqwdjwvkx44hw9nQfbwTur3z 36WZ7b592Q1sxsmr+GiVXeP14iH9DS+yuFJtTkTPdNz4+2B2eLAeiNfkw Q4sV4IoMhNxAB87s35+26k3kaIXcZaSHZbR8xZ7YlPU4JbU8NmSsB0aV1 A==; X-IronPort-AV: E=McAfee;i="6200,9189,10245"; a="247832199" X-IronPort-AV: E=Sophos;i="5.88,336,1635231600"; d="scan'208";a="247832199" Received: from orsmga008.jf.intel.com ([10.7.209.65]) by orsmga103.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 02 Feb 2022 01:48:49 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.88,336,1635231600"; d="scan'208";a="538182048" Received: from silpixa00401215.ir.intel.com ([10.55.128.96]) by orsmga008.jf.intel.com with ESMTP; 02 Feb 2022 01:48:48 -0800 From: Sean Morrissey <sean.morrissey@intel.com> To: Reshma Pattan <reshma.pattan@intel.com>, Stephen Hemminger <stephen@networkplumber.org> Cc: dev@dpdk.org, Sean Morrissey <sean.morrissey@intel.com> Subject: [PATCH v6 22/50] pcapng: remove unneeded header includes Date: Wed, 2 Feb 2022 09:47:34 +0000 Message-Id: <20220202094802.3618978-23-sean.morrissey@intel.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20220202094802.3618978-1-sean.morrissey@intel.com> References: <20220117201943.873922-1-sean.morrissey@intel.com> <20220202094802.3618978-1-sean.morrissey@intel.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions <dev.dpdk.org> List-Unsubscribe: <https://mails.dpdk.org/options/dev>, <mailto:dev-request@dpdk.org?subject=unsubscribe> List-Archive: <http://mails.dpdk.org/archives/dev/> List-Post: <mailto:dev@dpdk.org> List-Help: <mailto:dev-request@dpdk.org?subject=help> List-Subscribe: <https://mails.dpdk.org/listinfo/dev>, <mailto:dev-request@dpdk.org?subject=subscribe> Errors-To: dev-bounces@dpdk.org |
Series |
introduce IWYU
|
|
Checks
Context | Check | Description |
---|---|---|
ci/checkpatch | success | coding style OK |
Commit Message
Sean Morrissey
Feb. 2, 2022, 9:47 a.m. UTC
These header includes have been flagged by the iwyu_tool
and removed.
Signed-off-by: Sean Morrissey <sean.morrissey@intel.com>
---
lib/pcapng/rte_pcapng.c | 1 -
lib/pcapng/rte_pcapng.h | 2 --
2 files changed, 3 deletions(-)
Comments
On Wed, 2 Feb 2022 09:47:34 +0000 Sean Morrissey <sean.morrissey@intel.com> wrote: > These header includes have been flagged by the iwyu_tool > and removed. > > Signed-off-by: Sean Morrissey <sean.morrissey@intel.com> > --- > lib/pcapng/rte_pcapng.c | 1 - > lib/pcapng/rte_pcapng.h | 2 -- > 2 files changed, 3 deletions(-) > > diff --git a/lib/pcapng/rte_pcapng.c b/lib/pcapng/rte_pcapng.c > index 03edabe73e..5ae96a5bc9 100644 > --- a/lib/pcapng/rte_pcapng.c > +++ b/lib/pcapng/rte_pcapng.c > @@ -8,7 +8,6 @@ > #include <stdlib.h> > #include <string.h> > #include <sys/uio.h> > -#include <sys/utsname.h> > #include <time.h> > #include <unistd.h> yes, remove this. It was leftover from earlier code where utsname was called here in setup. > diff --git a/lib/pcapng/rte_pcapng.h b/lib/pcapng/rte_pcapng.h > index 8d3fbb1941..7d2697c647 100644 > --- a/lib/pcapng/rte_pcapng.h > +++ b/lib/pcapng/rte_pcapng.h > @@ -26,9 +26,7 @@ > #include <stdint.h> > #include <sys/types.h> > #include <rte_compat.h> > -#include <rte_common.h> > #include <rte_mempool.h> > -#include <rte_ring.h> > > #ifdef __cplusplus > extern "C" { Yes, these can go. Acked-by: Stephen Hemminger <stephen@networkplumber.org>
diff --git a/lib/pcapng/rte_pcapng.c b/lib/pcapng/rte_pcapng.c index 03edabe73e..5ae96a5bc9 100644 --- a/lib/pcapng/rte_pcapng.c +++ b/lib/pcapng/rte_pcapng.c @@ -8,7 +8,6 @@ #include <stdlib.h> #include <string.h> #include <sys/uio.h> -#include <sys/utsname.h> #include <time.h> #include <unistd.h> diff --git a/lib/pcapng/rte_pcapng.h b/lib/pcapng/rte_pcapng.h index 8d3fbb1941..7d2697c647 100644 --- a/lib/pcapng/rte_pcapng.h +++ b/lib/pcapng/rte_pcapng.h @@ -26,9 +26,7 @@ #include <stdint.h> #include <sys/types.h> #include <rte_compat.h> -#include <rte_common.h> #include <rte_mempool.h> -#include <rte_ring.h> #ifdef __cplusplus extern "C" {