murmurhash3 test failures on big-endian systems

Apollon Oikonomopoulos apoikos at debian.org
Tue Mar 27 11:31:36 EEST 2018


Hi,

On 12:55 Mon 26 Mar     , Josef 'Jeff' Sipek wrote:
> On Mon, Mar 26, 2018 at 15:57:01 +0300, Apollon Oikonomopoulos wrote:
> ...
> > I'd be happy to test the patch, thanks!
> 
> Ok, try the attached patch.  (It is a first pass at the issue, so it may not
> be the final diff that'll end up getting committed.  It'd be good to know if
> it actually fixes the issue for you - sadly, I don't have a big endian
> system to play with.)

Thanks for the quick response!

Unfortunately still fails, although with fewer assertion errors than 
before:

test-murmurhash3.c:34: Assert(#8) failed: memcmp(result, vectors[i].result, sizeof(result)) == 0
test-murmurhash3.c:34: Assert(#11) failed: memcmp(result, vectors[i].result, sizeof(result)) == 0
test-murmurhash3.c:34: Assert(#12) failed: memcmp(result, vectors[i].result, sizeof(result)) == 0
murmurhash3 (murmurhash3_32) ......................................... : FAILED
test-murmurhash3.c:34: Assert(#12) failed: memcmp(result, vectors[i].result, sizeof(result)) == 0
murmurhash3 (murmurhash3_128) ........................................ : FAILED

Regards,
Apollon


More information about the dovecot mailing list