> -----Original Message----- > From: Stephen Hemminger [mailto:[email protected]] > Sent: Thursday, August 24, 2017 7:37 PM > To: David Harton (dharton) <[email protected]> > Cc: [email protected]; [email protected]; > [email protected]; [email protected]; John Daley > (johndale) <[email protected]>; [email protected]; > [email protected]; [email protected]; > [email protected]; [email protected]; [email protected]; > [email protected]; [email protected]; > [email protected]; [email protected]; [email protected]; > [email protected]; [email protected]; [email protected]; > [email protected]; [email protected] > Subject: Re: [dpdk-dev] [PATCH] ethdev: modifiy vlan_offload_set_t to > return int > > On Thu, 24 Aug 2017 19:18:51 -0400 > David Harton <[email protected]> wrote: > > > @@ -2031,7 +2031,7 @@ struct avp_queue { > > mask = (ETH_VLAN_STRIP_MASK | > > ETH_VLAN_FILTER_MASK | > > ETH_VLAN_EXTEND_MASK); > > - avp_vlan_offload_set(eth_dev, mask); > > + (void)avp_vlan_offload_set(eth_dev, mask); > > This is a BSDism. You don't need the void cast.
Never know what to do...SA tools and some compilers whine if I don't. People complain if I do. :) What if I check the return code and log an error?
