Should we use SafeMath from OpenZeppelin?

[acuarica]

As mentioned in https://blog.openzeppelin.com/openzeppelin-contracts-4-0/

[...] Solidity 0.8 marks the end of the SafeMath era, as overflow checks are now built into the compiler.

You will find that SafeMath is still in the repository, athough it is now just a wrapper over the built-in overflow checks.

I see no point in use SafeMath. Am I missing something @fede-rodes?

[fede-rodes]

@acuarica agree, seems like there is no need for this library anymore

[acuarica]

We can remove TODO below then

renpool/contracts/RenPool.sol

Line 9 in 74e18f0

// TODO: use safeMath

[acuarica]

Migrated to new repo.