On Fri, Aug 25, 2017 at 12:55:21AM +0000, David Harton (dharton) wrote: > > > > -----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?
Sounds like a case of "doing the right thing" to me :-)
