onflow / cadence

Cadence, the resource-oriented smart contract programming language 🏃‍♂️
https://cadence-lang.org
Apache License 2.0
534 stars 138 forks source link

Remove unused Identifier() function from structs implementing atree.TypeInfo #3675

Closed fxamacker closed 2 weeks ago

fxamacker commented 2 weeks ago

Closes #3674

atree.TypeInfo interface used to include Identifier() function. However, Identifier() was removed later and is no longer part of atree.TypeInfo interface.

This PR removes Identifier() function from structs implementing atree.TypeInfo interface.


github-actions[bot] commented 2 weeks ago

Cadence Benchstat comparison

This branch with compared with the base branch onflow:master commit a56e52127ee2ff1036c8d6995ff4bc4ea7446874 The command for i in {1..N}; do go test ./... -run=XXX -bench=. -benchmem -shuffle=on; done was used. Bench tests were run a total of 7 times on each branch.

Collapsed results for better readability