Skip to content
New issue

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

Delete "Insecure Arithmetic" Page? #309

Open
DavidBDiligence opened this issue Nov 19, 2022 · 1 comment
Open

Delete "Insecure Arithmetic" Page? #309

DavidBDiligence opened this issue Nov 19, 2022 · 1 comment

Comments

@DavidBDiligence
Copy link
Contributor

Now that "Solidity automatically reverts on integer overflow and underflow, as of version 0.8.0." shall we delete the Insecure Arithmetic page?

@rupak21
Copy link

rupak21 commented Apr 4, 2023

@DavidBDiligence hey i think that even though Solidity now automatically reverts on integer overflow and underflow as of version 0.8.0, the Insecure Arithmetic page should not be deleted.

This is because the page still contains valuable information about how integer overflow and underflow can cause unexpected results in earlier versions of Solidity, as well as in other programming languages that do not have automatic overflow and underflow protection. In addition, there may be legacy code that is still being maintained and relies on the old behavior, so it is important to have this information available for those cases.

It is worth noting that the Insecure Arithmetic page should be updated to reflect the changes in Solidity 0.8.0, and to provide guidance on how to write code that is compatible with the new automatic overflow and underflow protection. This could include best practices for handling arithmetic operations and avoiding integer overflow and underflow, as well as information on how to handle errors that result from these operations.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants
@rupak21 @DavidBDiligence and others