On Thu, 2008-02-21 at 08:54 +0100, lorenm wrote: > +static const unsigned long pb_7f = 0x7f7f7f7f7f7f7f7fUL; > +static const unsigned long pb_80 = 0x8080808080808080UL; I think these should have a comment about truncation on 32-bit platforms being intentional. And the code will fail on a 128-bit platform :)