Yes, I will remove str_is function and submit the v3 of this patch soon.

On Sat, Nov 14, 2020 at 4:13 PM David Marchand <david.march...@redhat.com>
wrote:

> On Wed, Nov 11, 2020 at 9:17 AM Ibtisam Tariq <ibtisam.ta...@emumba.com>
> wrote:
> >
> > Instead of using getopt_long return value, strcmp was used to
> > compare the input parameters with the struct option array. This
> > patch get rid of all those strcmp by directly binding each longopt
> > with an int enum. This is to improve readability and consistency in
> > all examples.
> >
> > Bugzilla ID: 238
> > Cc: step...@networkplumber.org
> >
> > Reported-by: David Marchand <david.march...@redhat.com>
> > Signed-off-by: Ibtisam Tariq <ibtisam.ta...@emumba.com>
>
> This patch breaks compilation with clang:
> https://travis-ci.com/github/ovsrobot/dpdk/jobs/433053506#L1019
>
> You can remove this str_is helper.
>
>
> --
> David Marchand
>
>

-- 
- Ibtisam

Reply via email to