On Sun, Jun 06, 2010 at 02:16:24PM +1000, Peter Ross wrote: > + value <<= bits_per_plane; > + mask <<= bits_per_plane; You could align those two as well. No other comments from me...