I mailed a small CL for this.
It now says "uniform distribution on 64-bit integers".
That should imply reduction by masking (or other methods) is fine.

https://go-review.googlesource.com/c/go/+/219340


On Sunday, February 9, 2020 at 10:52:22 AM UTC-8, Juliusz Chroboczek wrote:
>
> It would be helpful if the introduction to the maphash package immediately 
> stated that it produces 64-bit values, and perhaps restate the fact that 
> the value can be safely reduced using bit masking (already there at the 
> Sum64 method). 
>
> I'm not sure what is the purpose of the BlockSize method -- why is the 
> block size relevant to the package user?  At the very least, the doc 
> should mention that Size is the value that the user wants. 
>
> -- Juliusz 
>

-- 
You received this message because you are subscribed to the Google Groups 
"golang-nuts" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to golang-nuts+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/golang-nuts/89c55493-f324-40cf-8832-e2487bb2a182%40googlegroups.com.

Reply via email to