Skip to content

Commit

Permalink
fix failing murmur tests on 32-bit platforms (apache#1008)
Browse files Browse the repository at this point in the history
* ensure number literal is uint64

* add myself to AUTHORS
  • Loading branch information
satta authored and Zariel committed Oct 28, 2017
1 parent 7fabcab commit 4c530e2
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
1 change: 1 addition & 0 deletions AUTHORS
Original file line number Diff line number Diff line change
Expand Up @@ -97,3 +97,4 @@ Nayef Ghattas <nayef.ghattas@datadoghq.com>
Michał Matczuk <mmatczuk@gmail.com>
Ben Krebsbach <ben.krebsbach@gmail.com>
Vivian Mathews <vivian.mathews.3@gmail.com>
Sascha Steinbiss <satta@debian.org>
4 changes: 2 additions & 2 deletions internal/murmur/murmur_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -66,8 +66,8 @@ func BenchmarkMurmur3H1(b *testing.B) {
b.RunParallel(func(pb *testing.PB) {
for pb.Next() {
h1 := Murmur3H1(data)
if h1 != 7627370222079200297 {
b.Fatalf("expected %d got %d", 7627370222079200297, h1)
if h1 != uint64(7627370222079200297) {
b.Fatalf("expected %d got %d", uint64(7627370222079200297), h1)
}
}
})
Expand Down

0 comments on commit 4c530e2

Please sign in to comment.