On 07/18/2014 08:54 AM, Jeffrey Deans wrote: > >> >>> + data &= ~(mask); \ >>> + data |= ((bits) & (mask)); \ >> >> Outer () not needed. > > Agreed. > Thanks for the review. This is a minor fix and I see no reason for v2. Ralf could you please remove the outer () when you get to apply this patch? Thanks! -- markos