[dpdk-dev] [PATCH] eal: fix Wbad-function-cast warning

Neil Horman nhorman at tuxdriver.com
Mon Mar 16 16:45:53 CET 2015


On Mon, Mar 16, 2015 at 03:04:45PM +0000, Mcnamara, John wrote:
> > -----Original Message-----
> > From: Neil Horman [mailto:nhorman at tuxdriver.com]
> > Sent: Monday, March 16, 2015 1:19 PM
> > To: Mcnamara, John
> > Cc: dev at dpdk.org
> > Subject: Re: [dpdk-dev] [PATCH] eal: fix Wbad-function-cast warning
> > 
> > 
> > This looks reasonable, but it rather begs the question as to why we need
> > rte_align_floor_int in the first place.  Theres only one other call site,
> > and it looks like it could use RTE_PTR_ALIGN_FLOOR just as easily.  What
> > about fixing up the second call site and removing the function to save
> > some space?
> 
> Hi Neil,
> 
> Seems like a good idea. I'll submit a v2.
> 
> Does rte_align_floor_int() need to be deprecated in some way or is it okay to just remove it?
> 
> John
> 
After the 2.0 release you should put a note in the api document, yes, but right
now its not required.  Though if you wanted to, I wouldn't stop you :)
Neil



More information about the dev mailing list