diff --git a/pkg/standalone-utils/contracts/relayer/BaseRelayerLibrary.sol b/pkg/standalone-utils/contracts/relayer/BaseRelayerLibrary.sol index 09ae033e33..3885a9929a 100644 --- 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.