Hi, On Fri, Sep 09, 2011, Johan Hedberg wrote: > On Fri, Sep 09, 2011, David Stockwell wrote: > > A short cut: the output is a contiguous array of uint_8, so masking > > with 0xFF should be unnecessary. Would this be required for non-x86 > > (including x86_64) architectures? > > Firstly, please don't top-post on this mailing list. > > If you don't do "& 0xff" and the value you're trying to assign to cid[n] > is greater than 0xff isn't the result of the integer overflow that > follows essentially the same as if you had done "% (0xff + 1)" (which is > not what you want in this case). Actually those two operations result in the same value (I even did a quick test program for this). So you're right :) Johan -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html