Summary

The ip_ variable is not calculated in the most gas-efficient way.

Vulnerability Details

By unwrapping the paranthesis of the expression ((endTime_ - payoutStart_ - intervalsPassedBefore_ * interval_) / interval_), the term intervalsPassedBefore would be first multiplied by interval_ only to be divided by it again. Removing this term from the paranthesis would avoid performing an additional multiplication operation.

Impact

Extra gas is spent for additional arithmetic operation

Tools Used

Manual inspection

Recommendations

Rewrite the term as ((endTime_ - payoutStart_) / interval_ - intervalsPassedBefore_)