Skip to content

Commit

Permalink
cli/reportgen
Browse files Browse the repository at this point in the history
  • Loading branch information
TilakMaddy committed Aug 2, 2024
1 parent 7f550ea commit 7a97cfa
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 26 deletions.
20 changes: 1 addition & 19 deletions reports/ccip-functions-report.md
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,6 @@ This report was generated by [Aderyn](https://github.com/Cyfrin/aderyn), a stati
- [L-11: Contract still has TODOs](#l-11-contract-still-has-todos)
- [L-12: Unused Custom Error](#l-12-unused-custom-error)
- [L-13: Loop contains `require`/`revert` statements](#l-13-loop-contains-requirerevert-statements)
- [L-14: Potentially unused `private` / `internal` state variables found.](#l-14-potentially-unused-private--internal-state-variables-found)


# Summary
Expand Down Expand Up @@ -102,7 +101,7 @@ This report was generated by [Aderyn](https://github.com/Cyfrin/aderyn), a stati
| Category | No. of Issues |
| --- | --- |
| High | 3 |
| Low | 14 |
| Low | 13 |


# High Issues
Expand Down Expand Up @@ -2451,20 +2450,3 @@ Avoid `require` / `revert` statements in a loop because a single bad item can ca



## L-14: Potentially unused `private` / `internal` state variables found.

State variable appears to be unused. No analysis has been performed to see if any inilne assembly references it. So if that's not the case, consider removing this unused variable.

<details><summary>1 Found Instances</summary>


- Found in src/v0.8/functions/v1_0_0/ocr/OCR2Base.sol [Line: 22](../tests/ccip-contracts/contracts/src/v0.8/functions/v1_0_0/ocr/OCR2Base.sol#L22)

```solidity
uint256 private constant maxUint32 = (1 << 32) - 1;
```

</details>



8 changes: 1 addition & 7 deletions reports/templegold-report.md
Original file line number Diff line number Diff line change
Expand Up @@ -8658,7 +8658,7 @@ Remove the redundant statements because no code will be generated and it just co

State variable appears to be unused. No analysis has been performed to see if any inilne assembly references it. So if that's not the case, consider removing this unused variable.

<details><summary>3 Found Instances</summary>
<details><summary>2 Found Instances</summary>


- Found in contracts/fakes/templegold/TempleGoldStakingMock.sol [Line: 71](../tests/2024-07-templegold/protocol/contracts/fakes/templegold/TempleGoldStakingMock.sol#L71)
Expand All @@ -8673,12 +8673,6 @@ State variable appears to be unused. No analysis has been performed to see if an
mapping(address account => EnumerableSet.UintSet indexes) private _accountStakes;
```

- Found in contracts/templegold/AuctionBase.sol [Line: 13](../tests/2024-07-templegold/protocol/contracts/templegold/AuctionBase.sol#L13)

```solidity
uint256 internal _currentEpochId;
```

</details>


Expand Down

0 comments on commit 7a97cfa

Please sign in to comment.