code review 13436045: runtime: Smhasher tests of our map hash function.

runtime: Smhasher tests of our map hash function.

[khr, 2013-09-06 21:10:51]

Hello golang-dev@googlegroups.com,

I'd like you to review this change to
https://code.google.com/p/go/

[rsc, 2013-09-06 22:08:23]

LGTM

[khr, 2013-09-06 23:23:49]

*** Submitted as https://code.google.com/p/go/source/detail?r=3338bbc9c09b ***

runtime: Smhasher tests of our map hash function.

R=golang-dev, rsc
CC=golang-dev
https://codereview.appspot.com/13436045

[Charlie Dorian, 2013-09-07 02:51:48]

On 2013/09/06 23:23:49, khr wrote:
> *** Submitted as https://code.google.com/p/go/source/detail?r=3338bbc9c09b ***
> 
> runtime: Smhasher tests of our map hash function.
> 
> R=golang-dev, rsc
> CC=golang-dev
> https://codereview.appspot.com/13436045

You could speed up, a little, hash_test.go: 

401 for c = 0.0; math.Pow(math.Erf(c/math.Sqrt(2)), float64(N)) < .99; c += .1 {
402 }

by replacing the function math.Sqrt(2) with the constant math.Sqrt2.

-- Charlie