Since the upcoming guardrails PR goes over the bytecode again, time to make some room. This is another pass, inspired by #746, that gives the Vault an even closer shave (saves 230 bytes).
Also updates the Vault bytecode script to show how many bytes it's over, when it is (since the data type kind of obscures it).
Type of change
[ ] Bug fix
[ ] New feature
[ ] Breaking change
[ ] Dependency changes
[ ] Code refactor / cleanup
[X] Optimization: [ ] gas / [X] bytecode
[ ] Documentation or wording changes
[ ] Other
Checklist:
[X] The diff is legible and has no extraneous changes
[X] Complex code has been commented, including external interfaces
[ ] Tests have 100% code coverage
[X] The base branch is either main, or there's a description of how to merge
Description
Since the upcoming guardrails PR goes over the bytecode again, time to make some room. This is another pass, inspired by #746, that gives the Vault an even closer shave (saves 230 bytes).
Also updates the Vault bytecode script to show how many bytes it's over, when it is (since the data type kind of obscures it).
Type of change
Checklist:
main, or there's a description of how to mergeIssue Resolution