Remove unnecessary package tweaks for crypto/rand.

nevkontakte · nevkontakte · commit 305cb64b7ddb · 2022-04-19T11:35:09.000+01:00
We could remove our overlay entirely, but our implementation seems to be
more advanced than the one of `js/wasm` in the standard library, so we
keep it for now.
diff --git a/build/context.go b/build/context.go
@@ -253,8 +253,6 @@ func (sc simpleCtx) applyPostloadTweaks(pkg *build.Package) *build.Package {
 		// GopherJS completely replaces sync.Pool implementation with a simpler one,
 		// since it always executes in a single-threaded environment.
 		pkg.GoFiles = exclude(pkg.GoFiles, "pool.go")
-	case "crypto/rand":
-		pkg.GoFiles = []string{"rand.go", "util.go"}
 	case "syscall/js":
 		// Reuse upstream tests to ensure conformance, but completely replace
 		// implementation.
diff --git a/compiler/natives/src/crypto/rand/rand.go b/compiler/natives/src/crypto/rand/rand.go
@@ -9,13 +9,9 @@ import (
 	"github.com/gopherjs/gopherjs/js"
 )
 
-func init() {
-	Reader = &rngReader{}
-}
-
-type rngReader struct{}
+type reader struct{}
 
-func (r *rngReader) Read(b []byte) (n int, err error) {
+func (r *reader) Read(b []byte) (n int, err error) {
 	array := js.InternalObject(b).Get("$array")
 	offset := js.InternalObject(b).Get("$offset").Int()
 
@@ -48,15 +44,3 @@ func (r *rngReader) Read(b []byte) (n int, err error) {
 
 	return 0, errors.New("crypto/rand not available in this environment")
 }
-
-func batched(f func([]byte) bool, readMax int) func([]byte) bool {
-	return func(buf []byte) bool {
-		for len(buf) > readMax {
-			if !f(buf[:readMax]) {
-				return false
-			}
-			buf = buf[readMax:]
-		}
-		return len(buf) == 0 || f(buf)
-	}
-}
