diff --git a/contracts/SafeProtocolRegistry.sol b/contracts/SafeProtocolRegistry.sol index 89dd11b9..0442aa44 100644 --- a/contracts/SafeProtocolRegistry.sol +++ b/contracts/SafeProtocolRegistry.sol @@ -119,7 +119,7 @@ contract SafeProtocolRegistry is ISafeProtocolRegistry, Ownable2Step { emit ModuleFlagged(module); } - function supportsInterface(bytes4 interfaceId) external view override returns (bool) { + function supportsInterface(bytes4 interfaceId) external pure override returns (bool) { return interfaceId == type(ISafeProtocolRegistry).interfaceId || interfaceId == type(IERC165).interfaceId; } } diff --git a/contracts/SignatureValidatorManager.sol b/contracts/SignatureValidatorManager.sol index 114bb44b..7f184e4a 100644 --- a/contracts/SignatureValidatorManager.sol +++ b/contracts/SignatureValidatorManager.sol @@ -223,7 +223,7 @@ contract SignatureValidatorManager is RegistryManager, ISafeProtocolFunctionHand * @param interfaceId bytes4 interface id to be checked * @return true if interface is supported */ - function supportsInterface(bytes4 interfaceId) external view override returns (bool) { + function supportsInterface(bytes4 interfaceId) external pure override returns (bool) { return interfaceId == type(IERC165).interfaceId || interfaceId == type(ISafeProtocolSignatureValidatorManager).interfaceId || diff --git a/contracts/test/TestFallbackReceiver.sol b/contracts/test/TestFallbackReceiver.sol index 56dc3a5c..bdd438cd 100644 --- a/contracts/test/TestFallbackReceiver.sol +++ b/contracts/test/TestFallbackReceiver.sol @@ -10,7 +10,7 @@ contract TestFallbackReceiver { receive() external payable { // solhint-disable-next-line no-unused-vars - (bool success, bytes memory data) = ethReceiver.call{value: address(this).balance}(""); + (bool success, ) = ethReceiver.call{value: address(this).balance}(""); if (!success) { revert("Failed to send eth"); } diff --git a/contracts/test/TestPlugin.sol b/contracts/test/TestPlugin.sol index 0c63bdbe..e68aa06f 100644 --- a/contracts/test/TestPlugin.sol +++ b/contracts/test/TestPlugin.sol @@ -18,7 +18,7 @@ abstract contract BaseTestPlugin is ISafeProtocolPlugin { permissions = _requiresPermission; } - function supportsInterface(bytes4 interfaceId) external view override returns (bool) { + function supportsInterface(bytes4 interfaceId) external pure override returns (bool) { return interfaceId == type(ISafeProtocolPlugin).interfaceId || interfaceId == 0x01ffc9a7; }