Core Contracts

Regnum Aurum Acquisition Corp
HardhatReal World AssetsNFT
77,280 USDC
View results
Submission Details
Severity: low
Invalid

[L-2] Simplifiable Calculation in `BaseGauge` Constructor

Description:
The nextPeriod variable of the constructor of BasGauge.sol is unnecessarily complex:

uint256 nextPeriod = ((currentTime / _periodDuration) * _periodDuration) + _periodDuration;

This calculation determines the next period's start time, but it can be simplified to:
currentTime + _periodDuration, improving readability and efficiency.

Impact:
The current formula performs unnecessary division and multiplication, making it less readable and slightly inefficient. While this does not introduce a critical bug, simplifying the calculation improves code clarity and reduces redundant operations.

Recommended Mitigation:
Use currentTime + _periodDuration instead.

Updates

Lead Judging Commences

inallhonesty Lead Judge 6 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.