Peter Hutterer wrote: > - nbytesRound = (nbytes + 3) & ~3; > + nbytesRound = pad_to_dwords(nbytes);
You're replacing two arithmetic operations with a function call here, and then ... > - rep->length = npixels + (nbytesRound >> 2); > + rep->length = npixels + num_dwords_for_bytes(nbytesRound); Is it really worth replacing a two-bit shift with a function call? _______________________________________________ xorg-devel mailing list [email protected] http://lists.x.org/mailman/listinfo/xorg-devel
