t11s
t11s
https://secureum.substack.com/p/smart-contract-security-101-secureum
https://github.com/runtimeverification/verified-smart-contracts/wiki/List-of-Security-Vulnerabilities
https://secureum.substack.com/p/security-pitfalls-and-best-practices-101
https://github.com/OpenZeppelin/openzeppelin-contracts/pull/3573/files is this version more optimized?
ugh yes this seems valid, idk why we made totalAssets account for fees
note to self: check with sam if this is safe (90% sure it is)
mind doing this against the `v7` branch instead of `master`? happy to merge this after resolving the comments above across the diff
prettier does this to us :(
thanks for opening an issue! i generally am not a fan of helper functions, tho this is one helper function that i think could be genuinely useful if you'd like...