Skip to content

Commit

Permalink
unique: not duplication implement cloneString
Browse files Browse the repository at this point in the history
Change-Id: I3af3bc01be3730172838a2fd4992e9db11cb6f37
  • Loading branch information
qiulaidongfeng committed May 4, 2024
1 parent ae6af9b commit 5169d26
Showing 1 changed file with 2 additions and 13 deletions.
15 changes: 2 additions & 13 deletions src/unique/clone.go
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@ package unique

import (
"internal/abi"
"internal/stringslite"
"unsafe"
)

Expand All @@ -20,7 +21,7 @@ import (
func clone[T comparable](value T, seq *cloneSeq) T {
for _, offset := range seq.stringOffsets {
ps := (*string)(unsafe.Pointer(uintptr(unsafe.Pointer(&value)) + offset))
*ps = cloneString(*ps)
*ps = stringslite.Clone(*ps)
}
return value
}
Expand Down Expand Up @@ -86,15 +87,3 @@ func buildArrayCloneSeq(typ *abi.Type, seq *cloneSeq, baseOffset uintptr) {
offset = (offset + align - 1) &^ (align - 1)
}
}

// cloneString is a copy of strings.Clone, because we can't depend on the strings
// package here. Several packages that might make use of unique, like net, explicitly
// forbid depending on unicode, which strings depends on.
func cloneString(s string) string {
if len(s) == 0 {
return ""
}
b := make([]byte, len(s))
copy(b, s)
return unsafe.String(&b[0], len(b))
}

0 comments on commit 5169d26

Please sign in to comment.