function arguments all start with a leading underscore
local vars that are used to cache storage variables (or otherwise would have a name collision) end in trailing underscores
prefix all revert messages with "UNIV2LPOracle/"
I only made those updates for the contract code, not the test file, because...just not worth the time. If I ever write the full test suite for this that I want to, I'll worry about test style then.
Does a few things:
uint-->uint256everywhereI only made those updates for the contract code, not the test file, because...just not worth the time. If I ever write the full test suite for this that I want to, I'll worry about test style then.