Heya,

+       {"FNV1a32", "fnv1a32", 30},
+       {"FNV1a64", "fnv1a64", 32},

I think those two should be FNV1A32/64, since they seem to be used to
define MHASH_* constants, and constants with lowercased letter are
really non-standard.

Other than that, as mentionned on irc, thanks for the patch, and I'd
be interested if you could run a hash benchmark [1] on it, if you have
it compiled, to see how it compares to other fast algos.

Cheers,
Jordi

[1] http://www.php.net/manual/en/function.hash.php#89574

-- 
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php

Reply via email to