On Thu, Feb 4, 2021 at 8:34 AM Ibtisam Tariq <[email protected]> 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: [email protected]
>
> Reported-by: David Marchand <[email protected]>
> Signed-off-by: Ibtisam Tariq <[email protected]>

For such a series, please provide a cover letter, this is easier when
replying on the whole series (and for me when applying).


For the series,
Reviewed-by: David Marchand <[email protected]>

There are still some unaligned examples (flow_classify, ip_reassembly,
kni, l2fwd-crypto, l3fwd-power, multi_process/symmetric_mp, vpda,
vmdq, vmdq_dcb), but this series is a good cleanup as is for now.

-- 
David Marchand

Reply via email to