Changing function visibility from public to external can save gas

[code423n4]

Handle

WatchPug

Vulnerability details

For public functions, the input parameters are copied to memory automatically which costs gas. If a function is only called externally, making its visibility as external will save gas because external functions’ parameters are not copied into memory and are instead read from calldata directly.

For example:

• depositRewardTokens()
• takeOutRewardTokens()
• redeemAllRewards()
• recoverUnstaking()
• unstake()
• stake()
• addValidator()
• disableValidator()
• setAllocatedTokensPerEpoch()
• setMaxCapMultiplier()
• setValidatorCommissionRate()
• setValidatorMinStakedRequired()
• redelegateUnstaked()
• transferUnstakedOut()

[kitti-katy]

duplicate of #2

[GalloDaSballo]

Duplicate of #2