Can we remove safe math ?

Cyfrin / solidity-by-example.github.io

Solidity By Example

https://solidity-by-example.org/

MIT License

614 stars 201 forks source link

Closed ihorbond closed 2 years ago

ihorbond commented 2 years ago

https://solidity-by-example.org/app/bi-directional-payment-channel/ specifies v0.8.13 but import SafeMath. Was it just left over from the pre 0.8.0 code or is it genuinely needed in this particular example ?

t4sk commented 2 years ago

It's not needed for 0.8

Can you submit a PR with the code change?

ihorbond commented 2 years ago

done @t4sk #237