UniswapV2Pair: reserves are using three different slots, not a single one

[Maksandre]

🐞 Bug Report

💻 Describe the bug

Because of the uint256 type, storage variables reserve0, reserve1, and blockTimestampLast are stored in three different slots, not a single one, as the comments state.

uniswap-v2-nil/contracts/UniswapV2Pair.sol

Lines 24 to 26 in 3ede108

	uint256 private reserve0; // uses single storage slot, accessible via getReserves
	uint256 private reserve1; // uses single storage slot, accessible via getReserves
	uint32 private blockTimestampLast; // uses single storage slot, accessible via getReserves

    uint256 private reserve0; // uses single storage slot, accessible via getReserves
    uint256 private reserve1; // uses single storage slot, accessible via getReserves
    uint32 private blockTimestampLast; // uses single storage slot, accessible via getReserves

💡 Possible Solution

Change types for reserve0 and reserve1 to uint112 or remove misleading comments