> >> -/**
> >> - * @internal
> >> - * Round up pointer 'x' to cache line size alignment
> >> + * Round up 'x' to alignment 'align'
> >>    */
> >> -#define ODP_CACHE_LINE_SIZE_ROUNDUP_PTR(x)\
> >> -       ((void *)ODP_CACHE_LINE_SIZE_ROUNDUP((uintptr_t)(x)))
> >> +#define ODP_ALIGN_ROUNDUP(x, align)\
> >> +       ((align) * (((x) + align - 1) / (align)))
> >>
> >
> Missing parens around 2nd instance of align here. should be
> ((align) * (((x) + (align) - 1)  / (align)))
> 
> 

This piece of code was moved inside the file. Anyway, I'll add the parens in v2 
while touching it.

-Petri

Reply via email to