Merge pull request #101 from ScaleChamp/master

Reduce unencodeValue allocations
go-stomp · May 9, 2021 · f6b94db · f6b94db
2 parents 68b55a5 + f65d066
commit f6b94db
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/frame/encode.go b/frame/encode.go
@@ -2,6 +2,7 @@ package frame
 
 import (
 	"strings"
+	"unsafe"
 )
 
 var (
@@ -19,9 +20,15 @@ var (
 	)
 )
 
+// Reduce one allocation on copying bytes to string
+func bytesToString(b []byte) string {
+	/* #nosec G103 */
+	return *(*string)(unsafe.Pointer(&b))
+}
+
 // Unencodes a header value using STOMP value encoding
 // TODO: return error if invalid sequences found (eg "\t")
 func unencodeValue(b []byte) (string, error) {
-	s := replacerForUnencodeValue.Replace(string(b))
+	s := replacerForUnencodeValue.Replace(bytesToString(b))
 	return s, nil
 }