reflect: fix incorrect nil copy #5
Conversation
|
不能复制 unsafe.Pointer 类型，因为现在的 copyPointer() 无法处理，我先把它忽略掉了 |
| @@ -144,7 +143,7 @@ func copyAny(src any, ptrs map[uintptr]any, copyConf *copyConfig) (dst any) { | |||
| dst = copyArray(src, ptrs, copyConf) | |||
| case reflect.Map: | |||
| dst = copyMap(src, ptrs, copyConf) | |||
| case reflect.Ptr, reflect.UnsafePointer: | |||
| case reflect.Ptr: | |||
There was a problem hiding this comment.
我看 UnsafePointer 不像 Ptr 类型一样可以推出它实际的类型，没办法用 reflect.New 新建一个(也许可以？)
大概只能像 Func 或 Int 类型一样，直接进行复制一份
不知道你是怎么决定这个值怎么复制的，我对反射不太了解，这里的判断如果不去掉，调用 copyPointer() 会报错的，所以就先去掉了 😄
There was a problem hiding this comment.
这是个好问题,确实之前没有考虑过unsafe.pointer的实际类型..
是不是能够把 unsafe.pointer 直接当成整数处理呢?
There was a problem hiding this comment.
个人认为是不可以的，因为是不正确的，例如 DeepCopy 把 UnsafePointer 当成 Int 复制一份:
a := []int{1, 2, 3}
ap := unsafe.Pointer(&a)
bp := DeepCopy(ap)
b := *(*[]int)(bp)
a[0] = 10
fmt.Println(a)
fmt.Println(b)
// [10 2 3]
// [10 2 3]那么就会出现这种非常常见的错误，所以复制它毫无意义，因为没有复制底层数据
当然可以当作无事发生，因为这种类型还是比较少见的，不过这样就不算 DeepCopy 了，如果要保证正确性的话，应该忽略处理它，直接报错
不过也许还有其他解决办法，比如现在的问题是 unsafe.Pointer 无法确认真正类型
但是如果把 copyAny 改为泛型，那么遇到 unsafe.Pointer 类型，可以再进行一次转换调用
这样就可以当作一个正常的 Ptrinter 类型走 copyPointer 流程处理了，例如判断 src 是 unsafe.Pointer 后可以：
dst = copyPointer((*T)(src), ptrs, copyConf)
这只是一个想法，我没有验证过
There was a problem hiding this comment.
唉 ，我最后说的方法应该不行，我想的太简单了...
因为 DeepCopy(unsafe.Porint(xxxx)) 时，参数就是 unsafe.Pointer 类型，根本没法推断，除非添加一个类型推断参数
但是如果复制结构字段时可不能指定类型，所以无法实现 😔
No description provided.