contracts/base and libraries: replace "if (A && B)" to save gas fees

[BurtonQin]

The solidity compiler generates more code for “if (A && B) {}” than for “if (A) {if (B) {} }”.
Our experiment shows that switching from “if (A && B) {}” to “if (A) {if (B) {} }” can save gas fee.

There are five such cases:

v3-periphery/contracts/base/PeripheryPayments.sol

Line 58 in 6cce88e

if (token == WETH9 && address(this).balance >= value) {

Replacing

if (token == WETH9 && address(this).balance >= value) {

with

if (token == WETH9) {
    if (address(this).balance >= value) {

can save gas fees and does not hurt code readability.

Similarly, the following code snippets can be improved:

v3-periphery/contracts/libraries/NFTDescriptor.sol

Line 224 in 6cce88e

if (params.decimalIndex > 0 && params.sigfigIndex == params.decimalIndex) {

v3-periphery/contracts/libraries/NFTDescriptor.sol

Line 277 in 6cce88e

if (difference > 0 && difference <= 18) {

v3-periphery/contracts/V3Migrator.sol

Line 78 in 6cce88e

if (params.refundAsETH && params.token0 == WETH9) {

v3-periphery/contracts/V3Migrator.sol

Line 91 in 6cce88e

if (params.refundAsETH && params.token1 == WETH9) {

[AbulKhayer2244]

#351 (comment)