Named return issue

[code423n4]

Handle

robee

Vulnerability details

Users can mistakenly think that the return value is the named return, but it is actually the actualreturn statement that comes after. To know that the user needs to read the code and is confusing.
Furthermore, removing either the actual return or the named return will save gas.

    BasketFacet.sol, calcTokensForAmount
    BasketFacet.sol, calcTokensForAmount
    BasketFacet.sol, calcTokensForAmountExit
    BasketFacet.sol, calcTokensForAmountExit

[0xleastwood]

This is non-critical.