[dpdk-dev] [PATCH 0/6] remove unncessary void cast

Ferruh Yigit ferruh.yigit at intel.com
Mon Sep 11 15:32:59 CEST 2017

On 9/11/2017 1:44 PM, Ferruh Yigit wrote:
> On 8/23/2017 4:44 PM, Stephen Hemminger wrote:
>> There are many places that cast away the return value from
>> rte_memcpy().  This was a common practice in the Unix BSD days because
>> lint would complain about unused return values, but it is not done
>> in modern Linux style because it clutters the code.
>> Maybe it was done for some Coverity warnings. If so, that issue
>> should be addressed with Coverity overrides not more code clutter.
>> Stephen Hemminger (6):
>>   testpmd: remove unnecessary void casts
>>   bnx2x: remove unnecssary void cast of rte_memcpy
>>   sfc: remove unnecessary cast of rte_memcpy
>>   e1000: remove unnecessary cast of rte_memcpy
>>   i40e: remove unnecessary cast of rte_memcpy
>>   ixgbe: remove unnecessary cast of rte_memcpy
> My concern is if coverity complains about return value not checked and
> someone starts fixing them adding void casts back.
> So I will evaluate the patchset for next-net, so that I can run coverity
> on them before merged into main tree, and see the result.

Coverity result on latest next-net (including this patchset)

rte_memcpy() return value not checked is not seen as an issue, so this
set is OK.

More information about the dev mailing list