From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail.droids-corp.org (zoll.droids-corp.org [94.23.50.67]) by dpdk.org (Postfix) with ESMTP id D120F5592 for ; Tue, 22 Nov 2016 14:59:41 +0100 (CET) Received: from lfbn-1-5996-232.w90-110.abo.wanadoo.fr ([90.110.195.232] helo=[192.168.1.13]) by mail.droids-corp.org with esmtpsa (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.84_2) (envelope-from ) id 1c9BfF-0000zJ-Lc; Tue, 22 Nov 2016 15:03:14 +0100 To: dev@dpdk.org, bruce.richardson@intel.com, pablo.de.lara.guarch@intel.com References: <1479822736-32399-1-git-send-email-olivier.matz@6wind.com> From: Olivier Matz Message-ID: <6f710163-fb03-a3dd-1520-ff78c07d146a@6wind.com> Date: Tue, 22 Nov 2016 14:59:35 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Icedove/45.4.0 MIME-Version: 1.0 In-Reply-To: <1479822736-32399-1-git-send-email-olivier.matz@6wind.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Subject: Re: [dpdk-dev] [PATCH 0/2] l2fwd/l3fwd: rework long options parsing X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 22 Nov 2016 13:59:41 -0000 Hi, On 11/22/2016 02:52 PM, Olivier Matz wrote: > These 2 patches were part of this RFC, which will not be integrated: > http://dpdk.org/ml/archives/dev/2016-September/046974.html > > It does not bring any functional change, it just reworks the way long > options are parsed in l2fwd and l3fwd to avoid uneeded strcmp() calls > and to ease the addition of a new long option in the future. > > I send them in case maintainers think it is better this way, but I have > no real need. > > Olivier Matz (2): > l3fwd: rework long options parsing > l2fwd: rework long options parsing > > examples/l2fwd/main.c | 30 +++++++-- > examples/l3fwd/main.c | 169 ++++++++++++++++++++++++++------------------------ > 2 files changed, 111 insertions(+), 88 deletions(-) > Sorry, I missed some checkpatch issues. I'll fix them in v2. I'm waiting a bit for other comments, in case of. Olivier