Assigned operations to constant variables

[code423n4]

Handle

pauliax

Vulnerability details

Impact

Assigned operations to constant variables are re-evaluated every time:

  bytes32 constant BASKET_STORAGE_POSITION =
        keccak256("diamond.standard.basket.storage");
  bytes32 constant CALL_STORAGE_POSITION =
        keccak256("diamond.standard.call.storage");
  bytes32 constant REENTRY_STORAGE_POSITION =
        keccak256("diamond.standard.reentry.storage");
  bytes32 constant ERC_20_STORAGE_POSITION =
        keccak256(
            // Compatible with pie-smart-pools
            "PCToken.storage.location"
        );

See ethereum/solidity#9232

Change from 'constant' to 'immutable'.

[0xleastwood]

Duplicate of #281