search

pontem-network / liquidswap

AMM core smart contracts
Other
167 stars 73 forks source link

Router v3 #124

Closed sunmoon11100 closed 1 year ago

sunmoon11100 commented 1 year ago

@borispovod How about including all math functions with overflow checking in math_v2 so that everyone can clearly understand all of its functions? The math module is only used in the router, so I think we can completely move it into math_v2.

borispovod commented 1 year ago

@sunmoon11100 i don't think it's good idea as it's also much more code to test/verify/audit, this why i keep only new functions.

borispovod commented 1 year ago

Published as optional router v3 in branch and updated docs with it.

borispovod commented 1 year ago

So closing for now as we keep it's changes optional.