Skip to content

Commit e798e4e

Browse files
author
divinerapier
committed
use std encoding lib
Signed-off-by: divinerapier <poriter.coco@gmail.com>
1 parent 41caf8a commit e798e4e

File tree

3 files changed

+90
-6
lines changed

3 files changed

+90
-6
lines changed

go.mod

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ require (
1212
github.com/disintegration/imaging v1.6.0
1313
github.com/dustin/go-humanize v1.0.0
1414
github.com/gabriel-vasile/mimetype v0.3.15
15+
github.com/go-logfmt/logfmt v0.4.0 // indirect
1516
github.com/go-redis/redis v6.15.2+incompatible
1617
github.com/go-sql-driver/mysql v1.4.1
1718
github.com/gocql/gocql v0.0.0-20190717234527-2ba2dd7440dc

weed/util/bytes.go

Lines changed: 27 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,10 @@
11
package util
22

3+
import "encoding/binary"
4+
35
// big endian
46

5-
func BytesToUint64(b []byte) (v uint64) {
7+
func BytesToUint64Old(b []byte) (v uint64) {
68
length := uint(len(b))
79
for i := uint(0); i < length-1; i++ {
810
v += uint64(b[i])
@@ -11,7 +13,7 @@ func BytesToUint64(b []byte) (v uint64) {
1113
v += uint64(b[length-1])
1214
return
1315
}
14-
func BytesToUint32(b []byte) (v uint32) {
16+
func BytesToUint32Old(b []byte) (v uint32) {
1517
length := uint(len(b))
1618
for i := uint(0); i < length-1; i++ {
1719
v += uint32(b[i])
@@ -20,26 +22,45 @@ func BytesToUint32(b []byte) (v uint32) {
2022
v += uint32(b[length-1])
2123
return
2224
}
23-
func BytesToUint16(b []byte) (v uint16) {
25+
func BytesToUint16Old(b []byte) (v uint16) {
2426
v += uint16(b[0])
2527
v <<= 8
2628
v += uint16(b[1])
2729
return
2830
}
29-
func Uint64toBytes(b []byte, v uint64) {
31+
func Uint64toBytesOld(b []byte, v uint64) {
3032
for i := uint(0); i < 8; i++ {
3133
b[7-i] = byte(v >> (i * 8))
3234
}
3335
}
34-
func Uint32toBytes(b []byte, v uint32) {
36+
func Uint32toBytesOld(b []byte, v uint32) {
3537
for i := uint(0); i < 4; i++ {
3638
b[3-i] = byte(v >> (i * 8))
3739
}
3840
}
39-
func Uint16toBytes(b []byte, v uint16) {
41+
func Uint16toBytesOld(b []byte, v uint16) {
4042
b[0] = byte(v >> 8)
4143
b[1] = byte(v)
4244
}
45+
46+
func BytesToUint64(b []byte) (v uint64) {
47+
return binary.BigEndian.Uint64(b)
48+
}
49+
func BytesToUint32(b []byte) (v uint32) {
50+
return binary.BigEndian.Uint32(b)
51+
}
52+
func BytesToUint16(b []byte) (v uint16) {
53+
return binary.BigEndian.Uint16(b)
54+
}
55+
func Uint64toBytes(b []byte, v uint64) {
56+
binary.BigEndian.PutUint64(b, v)
57+
}
58+
func Uint32toBytes(b []byte, v uint32) {
59+
binary.BigEndian.PutUint32(b, v)
60+
}
61+
func Uint16toBytes(b []byte, v uint16) {
62+
binary.BigEndian.PutUint16(b, v)
63+
}
4364
func Uint8toBytes(b []byte, v uint8) {
4465
b[0] = byte(v)
4566
}

weed/util/bytes_test.go

Lines changed: 62 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,62 @@
1+
package util
2+
3+
import (
4+
"reflect"
5+
"testing"
6+
)
7+
8+
// go test github.com/chrislusf/seaweedfs/weed/util -run '^(TestAll)$' -v
9+
10+
func TestAll(t *testing.T) {
11+
for i := 0; i <= 0xff; i += 7 {
12+
assertU16(t, uint16(i))
13+
}
14+
for i := 0; i <= 0xffff; i += 211 {
15+
assertU32(t, uint32(i))
16+
}
17+
for i := 0; i <= 0xffffffff; i += 1000000007 {
18+
assertU64(t, uint64(i))
19+
}
20+
}
21+
22+
func assertU16(t *testing.T, i uint16) {
23+
var buf1, buf2 [2]byte
24+
Uint16toBytesOld(buf1[:], i)
25+
Uint16toBytes(buf2[:], i)
26+
if !reflect.DeepEqual(buf1, buf2) {
27+
t.Errorf("i: %d, buf1: %v, buf2: %v\n", i, buf1, buf2)
28+
}
29+
v1 := BytesToUint16Old(buf1[:])
30+
v2 := BytesToUint16(buf2[:])
31+
if v1 != v2 {
32+
t.Errorf("buf1: %v, v1: %d, buf2: %v, v2: %d\n", buf1, v1, buf2, v2)
33+
}
34+
}
35+
36+
func assertU32(t *testing.T, i uint32) {
37+
var buf1, buf2 [4]byte
38+
Uint32toBytesOld(buf1[:], i)
39+
Uint32toBytes(buf2[:], i)
40+
if !reflect.DeepEqual(buf1, buf2) {
41+
t.Errorf("i: %d, buf1: %v, buf2: %v\n", i, buf1, buf2)
42+
}
43+
v1 := BytesToUint32Old(buf1[:])
44+
v2 := BytesToUint32(buf2[:])
45+
if v1 != v2 {
46+
t.Errorf("buf1: %v, v1: %d, buf2: %v, v2: %d\n", buf1, v1, buf2, v2)
47+
}
48+
}
49+
50+
func assertU64(t *testing.T, i uint64) {
51+
var buf1, buf2 [8]byte
52+
Uint64toBytesOld(buf1[:], i)
53+
Uint64toBytes(buf2[:], i)
54+
if !reflect.DeepEqual(buf1, buf2) {
55+
t.Errorf("i: %d, buf1: %v, buf2: %v\n", i, buf1, buf2)
56+
}
57+
v1 := BytesToUint64Old(buf1[:])
58+
v2 := BytesToUint64(buf2[:])
59+
if v1 != v2 {
60+
t.Errorf("buf1: %v, v1: %d, buf2: %v, v2: %d\n", buf1, v1, buf2, v2)
61+
}
62+
}

0 commit comments

Comments
 (0)