There is one irrelevant commit in this PR: removing require(ts == block.timestamp from seek. A quick read of the Uniswap v2 code is sufficient to show this is useless at best: https://github.com/Uniswap/uniswap-v2-core/blob/master/contracts/UniswapV2Pair.sol . OTOH, removing it will save a little gas. I slipped it into this PR since I was touching seek anyway, but happy to move it to its own PR if that is preferred.
See updated comments for explanation.
There is one irrelevant commit in this PR: removing
require(ts == block.timestampfromseek. A quick read of the Uniswap v2 code is sufficient to show this is useless at best: https://github.com/Uniswap/uniswap-v2-core/blob/master/contracts/UniswapV2Pair.sol . OTOH, removing it will save a little gas. I slipped it into this PR since I was touchingseekanyway, but happy to move it to its own PR if that is preferred.