Re: [dpdk-dev] [v2 PATCH 3/8] net/ipn3ke: fix incorrect commit check logic

2019-11-08 Thread Xu, Rosen
Hi, > -Original Message- > From: Kevin Traynor [mailto:ktray...@redhat.com] > Sent: Thursday, November 07, 2019 3:02 > To: dev@dpdk.org > Cc: david.march...@redhat.com; Kevin Traynor ; > Xu, Rosen ; sta...@dpdk.org > Subject: [v2 PATCH 3/8] net/ipn3ke: fix incorrect commit check logic > >

[dpdk-dev] [v2 PATCH 3/8] net/ipn3ke: fix incorrect commit check logic

2019-11-06 Thread Kevin Traynor
Coverity is complaining about identical code regardless of which branch of the if else is taken. Functionally it means an error will always be returned if this if else is hit. Remove the else branch. CID 337928 (#1 of 1): Identical code for different branches (IDENTICAL_BRANCHES)identical_