On Tue, Jul 1, 2014 at 4:30 AM, Phil Carmody <phil@xxxxxxxxxx> wrote: > > With unsigned, it becomes a lot quieter, but still much noiser than my > original. (I have a v3 which excludes an idiom using '/'.) > Yes, because it cover more than just binary operations. If you consider zero extend the unsigned type introducing error, the binary operation just how you use the error bits. >> #define NLMSG_ALIGNTO 4U > > I consider that, pedantically, to be worth fixing. alignments are defined > by the C standard to be of type size_t. It should either be ((size_t)4) ((size_t)4) is hell ugly. > if you care about the precise type, or 4 if you don't, but not 4U. The > 'U' adds nothing apart from a way to create an incorrect mask. That is debatable. Enforcing 4U vs 4 is annoying and buy you nothing most of the case. I don't consider alignment being 4U as wrong by itself. The key question is how you use it. if we introduce: #define ALIGNMASK(align) (size_t)(~((align) -1)) And we always use the macro to create mask from alignment value. Then there is no need to enforce signedness of the var passed into the macro. Hey, it might even read better. Chris -- To unsubscribe from this list: send the line "unsubscribe linux-sparse" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
