feat: update SafeErc20 to newer Solidity version #652
+417
−111
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…zero-reset strategy
…in ISafeErc20 trait
…xed method for clarity and consistency in ISafeErc20 trait
…contract implementation
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
…mon0x1800/OpenZeppelin-rust-contracts-stylus into feature/620-update-safe-erc20
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
…g with calldata for improved error handling and clarity
…mon0x1800/OpenZeppelin-rust-contracts-stylus into feature/620-update-safe-erc20
…proved clarity and consistency
✅ Deploy Preview for contracts-stylus canceled.
|
1 task
bidzyyys
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR updates the SafeERC20 implementation to match the latest Solidity version, adding support for new features and improving error handling. The changes include:
try_safe_transferandtry_safe_transfer_fromfunctions that return bool instead of revertingforce_approvefunction for handling tokens requiring zero-reset approvaltransfer_and_call_relaxed,transfer_from_and_call_relaxed, andapprove_and_call_relaxedfunctionssafe_increase_allowanceandsafe_decrease_allowancefunctionsResolves #620
PR Checklist