Manager Static Analysis Findings

MAN-01S: Dead Code

Description:

The linked variables remain unutilized within the contract.

Example:

Copy
33uint256 public currentCycle;
34uint256 public currentCycleIndex;
35uint256 public deployedBlock;
36uint256 public currentlyDeployingCycle;
37uint256 public cycleDuration;

Recommendation:

We advise them to be omitted from the codebase.

Alleviation:

Both variables were safely omitted from the codebase.

MAN-02S: Literal Boolean Comparison

Description:

The linked statement performs a literal comparison of a bool variable and the value of true.

Example:

Copy
148if (params.complete == true) {
149    _completeRollover(params.rewardsIpfsHash);
150}

Recommendation:

We advise the variable to be used directly as it already represents a bool.

Alleviation:

The literal comparison was replaced by the raw utilization of the variable.