internal
functions can be private
#70
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
Handle
pants
Vulnerability details
These
internal
functions are never called in any contract that inherits their contract:Swap.fillZrxQuote()
Swap.signifiesETHOrZero()
EmergencyGovernable.governor()
Therefore, their visibility can be reduced to
private
.Impact
private
functions are cheaper thaninternal
functions.Tool Used
Manual code review.
Recommended Mitigation Steps
Define these functions as
private
.The text was updated successfully, but these errors were encountered: