Use pointer receivers for some types:

- Trie `Node`'s `Copy` method
- pprof `Settings`'s `String` method
Keep value receivers for some types:
- Keep value receivers for VDTs
- `common.Hash`, `transaction.priorityQueue`

internal/pprof/settings.go

@@ -18,13 +18,13 @@ type Settings struct {
 	MutexProfileRate int
 }
 
-func (s *Settings) setDefaults() { //skipcq: GO-W1029
+func (s *Settings) setDefaults() {
 	if s.ListeningAddress == "" {
 		s.ListeningAddress = "localhost:6060"
 	}
 }
 
-func (s Settings) String() string { //skipcq: GO-W1029
+func (s *Settings) String() string {
 	return fmt.Sprintf(
 		"listening on %s and setting block profile rate to %d, mutex profile rate to %d",
 		s.ListeningAddress, s.BlockProfileRate, s.MutexProfileRate)

internal/trie/node/copy.go

@@ -53,7 +53,7 @@ type CopySettings struct {
 // Copy deep copies the node.
 // Setting copyChildren to true will deep copy
 // children as well.
-func (n *Node) Copy(settings CopySettings) *Node { //skipcq: GO-W1029
+func (n *Node) Copy(settings CopySettings) *Node {
 	cpy := &Node{
 		Dirty:       n.Dirty,
 		Generation:  n.Generation,

pkg/scale/varying_data_type.go

@@ -83,7 +83,7 @@ func (vdt *VaryingDataType) Value() (VaryingDataTypeValue, error) { //skipcq: GO
 	return vdt.value, nil
 }
 
-func (vdt VaryingDataType) String() string { //skipcq: GO-W1029
+func (vdt *VaryingDataType) String() string { //skipcq: GO-W1029
 	if vdt.value == nil {
 		return "VaryingDataType(nil)"
 	}