Simon Josefsson <[EMAIL PROTECTED]> writes: >> -#define rol(x,n) ( ((x) << (n)) | ((x) >> (32-(n))) ) >> +#define rol(x, n) (((x) << (n)) | ((uint32_t) (x) >> (32 - (n)))) > ... >> -#define rol(x, n) (((x) << (n)) | ((x) >> (32 - (n)))) >> +#define rol(x, n) (((x) << (n)) | ((uint32_t) (x) >> (32 - (n)))) > > I don't understand this. Why would a left-shifted uint32_t become any > other type?
Oops, I didn't read your ChangeLog entry.... I've installed all of your patch now. Thanks, Simon
