We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
For the beta and mainnet launch, per the call for audits it has been requested that tm2 also be audited but while doing so I noticed this code https://github.com/gnolang/gno/blob/master/tm2/pkg/cmap/cmap.go in which really it is just a map with a mutex but doesn't account for load characteristics and content the way that Go's standard library https://pkg.go.dev/sync#Map
Perhaps let us delete the code in cmap and instead use sync.Map
Kindly paging @n2p5 @moul @petar-dambovaliev
The text was updated successfully, but these errors were encountered:
Agreed, let's change it to a sync.Map
Sorry, something went wrong.
No branches or pull requests
For the beta and mainnet launch, per the call for audits it has been requested that tm2 also be audited but while doing so I noticed this code https://github.com/gnolang/gno/blob/master/tm2/pkg/cmap/cmap.go in which really it is just a map with a mutex but doesn't account for load characteristics and content the way that Go's standard library https://pkg.go.dev/sync#Map
Perhaps let us delete the code in cmap and instead use sync.Map
Kindly paging @n2p5 @moul @petar-dambovaliev
The text was updated successfully, but these errors were encountered: