Gas Optimizations #101
Labels
bug
Something isn't working
G (Gas Optimization)
sponsor acknowledged
Technically the issue is correct, but we're not going to resolve it for XYZ reasons
Every reason string takes at least 32 bytes. Use short reason strings that fits in 32 bytes or it will become more expensive.
##Tool Used
Manual Review
##Occurance :
https://github.com/code-423n4/2022-04-backed/blob/e8015d7c4b295af131f017e646ba1b99c8f608f0/contracts/NFTLoanFacilitator.sol#L81-L86
Sometimes using mutiple require can save more gas, but in this case it was cost less gas than using mutiple require. So this implementation below can saving more gas
##Tool Used
yarn test & remix
##Recommended Mitigation
change to
The text was updated successfully, but these errors were encountered: