revert: change _TEMP_STORAGE_SUFFIX to immutable

see: ethereum/solidity#9232
balancer · Oct 27, 2021 · ed64d91 · ed64d91
1 parent fb7f8fc
commit ed64d91
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/pkg/standalone-utils/contracts/relayer/BaseRelayerLibrary.sol b/pkg/standalone-utils/contracts/relayer/BaseRelayerLibrary.sol
@@ -96,9 +96,9 @@ contract BaseRelayerLibrary is IBaseRelayerLibrary {
         }
     }
 
-    bytes32 private constant _TEMP_STORAGE_SUFFIX = keccak256("balancer.base-relayer-library");
+    bytes32 private immutable _TEMP_STORAGE_SUFFIX = keccak256("balancer.base-relayer-library");
 
-    function _getTempStorageSlot(uint256 ref) private pure returns (bytes32) {
+    function _getTempStorageSlot(uint256 ref) private view returns (bytes32) {
         // This replicates the mechanism Solidity uses to allocate storage slots for mappings, but using a hash as the
         // mapping's storage slot, and subtracting 1 at the end. This should be more enough to prevent collisions with
         // other state variables this or derived contracts might use.