From mboxrd@z Thu Jan  1 00:00:00 1970
From: Thomas Monjalon <thomas.monjalon@6wind.com>
Subject: Re: [PATCH 0/2] l2fwd/l3fwd: rework long options parsing
Date: Tue, 17 Jan 2017 18:11:04 +0100
Message-ID: <1632514.EAEELXtbXD@xps13>
References: <1479822736-32399-1-git-send-email-olivier.matz@6wind.com>
 <6f710163-fb03-a3dd-1520-ff78c07d146a@6wind.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7Bit
Cc: dev@dpdk.org, bruce.richardson@intel.com, pablo.de.lara.guarch@intel.com
To: Olivier Matz <olivier.matz@6wind.com>
Return-path: <dev-bounces@dpdk.org>
Received: from mail-lf0-f49.google.com (mail-lf0-f49.google.com
 [209.85.215.49]) by dpdk.org (Postfix) with ESMTP id 74D551094
 for <dev@dpdk.org>; Tue, 17 Jan 2017 18:11:07 +0100 (CET)
Received: by mail-lf0-f49.google.com with SMTP id k86so113347744lfi.0
 for <dev@dpdk.org>; Tue, 17 Jan 2017 09:11:07 -0800 (PST)
In-Reply-To: <6f710163-fb03-a3dd-1520-ff78c07d146a@6wind.com>
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <http://dpdk.org/ml/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://dpdk.org/ml/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <http://dpdk.org/ml/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
Errors-To: dev-bounces@dpdk.org
Sender: "dev" <dev-bounces@dpdk.org>

2016-11-22 14:59, Olivier Matz:
> 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.

There was no comment.
It seems this version is good enough.

Applied with a checkpatch fix, thanks