-
Notifications
You must be signed in to change notification settings - Fork 838
Open
ava-labs/coreth
#1347Description
Can we unify all of these implementations?
- https://github.com/ava-labs/coreth/blob/0a6ce77563f1865b0a05f8d507bb6a4ba222aa26/plugin/evm/customtypes/header_ext_test.go#L192
- https://github.com/ava-labs/coreth/blob/0a6ce77563f1865b0a05f8d507bb6a4ba222aa26/utils/numbers.go#L11
I think a single:
func PointerTo[T any](x T) *T {
return &x
}should work for all our usages.
Originally posted by @StephenButtolph in ava-labs/coreth#1263 (comment)
NewUint64 has itself 173 references. I changed others but we should probably change that in another PR