From mboxrd@z Thu Jan 1 00:00:00 1970 From: Gaetan Rivet Subject: [PATCH v1 0/8] Clean up rte_option Date: Thu, 20 Dec 2018 18:06:39 +0100 Message-ID: Mime-Version: 1.0 Content-Transfer-Encoding: 8bit Cc: Gaetan Rivet To: dev@dpdk.org Return-path: Received: from mail-wm1-f67.google.com (mail-wm1-f67.google.com [209.85.128.67]) by dpdk.org (Postfix) with ESMTP id 1FEFE1BBD6 for ; Thu, 20 Dec 2018 18:07:15 +0100 (CET) Received: by mail-wm1-f67.google.com with SMTP id n190so2825126wmd.0 for ; Thu, 20 Dec 2018 09:07:15 -0800 (PST) List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" rte_option lib might benefit from a few improvements. Overview: - list registered options on /rte/app --help - check duplicates within base EAL options - rename some fields to be closer to standard struct option and just cleaner overall. - fix some strange code (static iterator?) - fix doc, improve API a little... Gaetan Rivet (8): option: use bare option string as name option: do not use static iterator option: add usage string option: rename name field option: check against common option on register option: fix log level of error in register option: improve library documentation option: return an error when register fail lib/librte_eal/common/eal_common_options.c | 1 + lib/librte_eal/common/eal_private.h | 7 +++ lib/librte_eal/common/include/rte_option.h | 17 +++++-- lib/librte_eal/common/rte_option.c | 57 +++++++++++++++++++--- lib/librte_telemetry/rte_telemetry.c | 3 +- 5 files changed, 72 insertions(+), 13 deletions(-) -- 2.19.1