search

beefyfinance / beefy-contracts

Public repo for the community devs to advance the Beefy protocol.
https://app.beefy.finance
175 stars 163 forks source link

Use internal function in onlyManager modifier #233

Closed roman-monk closed 1 year ago

roman-monk commented 1 year ago

had to deal with contract size limit recently and changing onlyManager modifier to function can save up to 590 bytes, which is like 2.5% of total limit. implemented as in OZ onlyOwner/whenNotPaused

MirthFutures commented 1 year ago

Looks good, had to do this as well before. Nice little contract sizing trick.

kexleyBeefy commented 1 year ago

Looks good to me too.