cleanup

Author: d10r

packages/ethereum-contracts/contracts/superfluid/SuperToken.sol

@@ -102,7 +102,6 @@ contract SuperToken is
     // function in its respective mock contract to ensure that it doesn't break anything or lead to unexpected
     // behaviors/layout when upgrading
 
-    //uint256 internal _reserve22;
     uint256 internal _reserve23;
     uint256 private _reserve24;
     uint256 private _reserve25;
@@ -283,7 +282,7 @@ contract SuperToken is
     /// @dev EIP-712 Domain Separator
     // solhint-disable func-name-mixedcase
     function DOMAIN_SEPARATOR() public view virtual override returns (bytes32) {
-        // TODO: can be optimized: provide immutable parts from constants
+        // Here we could squeeze out some gas by using pre-computed hashes
         return keccak256(
             abi.encode(
                 _EIP712_DOMAIN_TYPEHASH,

packages/ethereum-contracts/foundry.toml

@@ -15,7 +15,6 @@ remappings = [
     '@superfluid-finance/ethereum-contracts/contracts/=packages/ethereum-contracts/contracts/',
     '@superfluid-finance/solidity-semantic-money/src/=packages/solidity-semantic-money/src/',
     '@openzeppelin/contracts/=node_modules/@openzeppelin/contracts/',
-    '@openzeppelin/contracts-v5/=node_modules/@openzeppelin/contracts-v5/',
     'ds-test/=lib/forge-std/lib/ds-test/src/',
     'forge-std/=lib/forge-std/src/']
 out = 'packages/ethereum-contracts/build/foundry/default'