llamaxyz / llama

Llama is an onchain governance and access control framework for smart contracts.
https://llama.xyz
MIT License
47 stars 5 forks source link

refactor: `uint256` -> `uint96` in checkpoint libs. #464

Closed 0xrajath closed 1 year ago

0xrajath commented 1 year ago

https://github.com/spearbit-audits/review-llama2/pull/10#discussion_r1276785364

TO BE DISCUSSED: Use uint96 instead of uint256 in the Checkpoint libs since only uint96 are passed into them.

AustinGreen commented 1 year ago

We chose to acknowledge but close this issue. Leaving it as a uint256 minimizing the diff between the original OZ library.