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

subやmulなのではなく単純に足し算引き算に変えるのはいかがでしょうか? #484

Open
3tomcha opened this issue Dec 24, 2023 · 2 comments

Comments

@3tomcha
Copy link

3tomcha commented Dec 24, 2023

提案内容

subやmulなのではなく単純に足し算引き算に変えるのはいかがでしょうか?

背景

お世話になっております!
solidityの新しいバージョンでは、SafeMathの機能が言語レベルで組み込まれており、subやmulなのではなく単純に足し算引き算などができるようになりました
https://docs.soliditylang.org/en/v0.8.23/080-breaking-changes.html

備考

https://app.unchain.tech/learn/AVAX-AMM/ja/1/3/

@3tomcha
Copy link
Author

3tomcha commented Dec 24, 2023

方針的に問題なければ私の方でPR出せます!
ご検討よろしくお願いします!

@neila
Copy link
Member

neila commented Dec 24, 2023

ありがとうございます!仰る通りですね、safeMathに対応しましょう!PRお待ちしています 🥳

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

2 participants