modifier canAddLiquidity and function _canAddLiquidity

[code423n4]

Handle

pauliax

Vulnerability details

Impact

modifier canAddLiquidity calls internal function _canAddLiquidity. This function is not called anywhere else so I do not see a reason why all the logic can't be moved to the modifier to save some gas by reducing the extra call.

Recommended Mitigation Steps

Remove function _canAddLiquidity, place its logic directly in the canAddLiquidity modifier.

[asselstine]

I believe our intention was to have an external function to check; so in fact the fix here would be to add the external function

canAddLiquidity