We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
These variables are only able to be set in the constructor. Making them immutable will save gas when reading.
https://github.com/code-423n4/2022-05-cudos/blob/de39cf3cd1f1e1cf211819b06d4acf6a043acda0/solidity/contracts/Gravity.sol#L59-L61
https://github.com/code-423n4/2022-05-cudos/blob/de39cf3cd1f1e1cf211819b06d4acf6a043acda0/solidity/contracts/CosmosToken.sol#L5
All for loops found in gravity.sol can be optimized by incrementing i within unchecked{}. The new pattern would be:
gravity.sol
i
unchecked{}
for (uint i = 0; i < length;) { doStuff(); unchecked { ++i; } }
https://github.com/code-423n4/2022-05-cudos/blob/de39cf3cd1f1e1cf211819b06d4acf6a043acda0/solidity/contracts/Gravity.sol#L453-L456
The text was updated successfully, but these errors were encountered:
0xDjango issue #92
78df874
No branches or pull requests
Optimization #1 - These variables can be set to immutable
These variables are only able to be set in the constructor. Making them immutable will save gas when reading.
https://github.com/code-423n4/2022-05-cudos/blob/de39cf3cd1f1e1cf211819b06d4acf6a043acda0/solidity/contracts/Gravity.sol#L59-L61
Optimization #2 - This variable can be set to CONSTANT
https://github.com/code-423n4/2022-05-cudos/blob/de39cf3cd1f1e1cf211819b06d4acf6a043acda0/solidity/contracts/CosmosToken.sol#L5
Optimization #3 - For loop optimization
All for loops found in
gravity.sol
can be optimized by incrementingi
withinunchecked{}
. The new pattern would be:https://github.com/code-423n4/2022-05-cudos/blob/de39cf3cd1f1e1cf211819b06d4acf6a043acda0/solidity/contracts/Gravity.sol#L453-L456
The text was updated successfully, but these errors were encountered: