Update to go 1.20

.github/workflows/go.yml

@@ -1,8 +1,5 @@
 name: Go
 
-env:
-  go-version: "1.19"
-
 on:
   pull_request:
   push:
@@ -17,7 +14,8 @@ jobs:
       - name: set up go
         uses: actions/setup-go@v4
         with:
-          go-version: ${{ env.go-version }}
+          check-latest: true
+          go-version-file: "go.mod"
       - name: fmt, tidy
         run: |
           make install
@@ -35,7 +33,8 @@ jobs:
       - name: set up go
         uses: actions/setup-go@v4
         with:
-          go-version: ${{ env.go-version }}
+          check-latest: true
+          go-version-file: "go.mod"
       - name: setup env
         run: make install
       - name: lint
@@ -57,7 +56,8 @@ jobs:
       - name: set up go
         uses: actions/setup-go@v4
         with:
-          go-version: ${{ env.go-version }}
+          check-latest: true
+          go-version-file: "go.mod"
       - name: setup env
         run: make install
       - name: Clear test cache

go.mod

@@ -1,6 +1,6 @@
 module github.com/spacemeshos/go-scale
 
-go 1.19
+go 1.20
 
 require (
 	github.com/google/go-cmp v0.5.9

unsafe.go

@@ -1,24 +1,24 @@
 package scale
 
 import (
-	"reflect"
 	"unsafe"
 )
 
 // stringToBytes converts a string to a byte slice without copying the underlying data.
 // IMPORTANT: The returned byte slice must not be modified!
 // This is a low-level function and should be used carefully.
 func stringToBytes(s string) (b []byte) {
-	bh := (*reflect.SliceHeader)(unsafe.Pointer(&b))
-	sh := (*reflect.StringHeader)(unsafe.Pointer(&s))
-	bh.Data = sh.Data
-	bh.Cap = sh.Len
-	bh.Len = sh.Len
-	return b
+	if s == "" {
+		return nil
+	}
+	return unsafe.Slice(unsafe.StringData(s), len(s))
 }
 
 // bytesToString converts a byte slice to a string without copying the underlying data.
 // This is a low-level function and should be used carefully.
 func bytesToString(b []byte) string {
-	return *(*string)(unsafe.Pointer(&b))
+	if len(b) == 0 {
+		return ""
+	}
+	return unsafe.String(unsafe.SliceData(b), len(b))
 }