diff --git a/contracts/instances/ERC20TornadoCloneable.sol b/contracts/instances/ERC20TornadoCloneable.sol index 20c609f..8947ffa 100644 --- a/contracts/instances/ERC20TornadoCloneable.sol +++ b/contracts/instances/ERC20TornadoCloneable.sol @@ -63,7 +63,7 @@ contract ERC20TornadoCloneable is ERC20Tornado { address _token, uint256 _denomination, uint32 _merkleTreeHeight - ) external { + ) public virtual { // Part of ERC20TornadoCloneable.sol, denomination and levels is obviously // in all older code set once and never changed again, as such, we can use // these for `initialize`. The question comes, why is it set to 1 above? @@ -107,7 +107,7 @@ contract ERC20TornadoCloneable is ERC20Tornado { system which will require the relayers to have a wallet-like smart contract, because then frontends will be able to build proofs for it, and as such there will be no need for this bullshit. */ - function checkInfrastructureIsDead() external { + function checkInfrastructureIsDead() public virtual { require(router != address(0), "infrastructure already dead"); try @@ -160,7 +160,7 @@ contract ERC20TornadoCloneable is ERC20Tornado { address payable _relayer, uint256 _fee, uint256 _refund - ) internal override { + ) internal virtual override { // This is the part which we don't check if infra is down if (router != address(0)) { require(msg.sender == router, "if infrastructure not dead, router must be caller"); diff --git a/contracts/instances/ETHTornadoCloneable.sol b/contracts/instances/ETHTornadoCloneable.sol index b68cb2a..b69a609 100644 --- a/contracts/instances/ETHTornadoCloneable.sol +++ b/contracts/instances/ETHTornadoCloneable.sol @@ -28,7 +28,7 @@ contract ETHTornadoCloneable is ETHTornado { * @param _merkleTreeHeight The merkle tree height of the instance. Usually 20. */ - function initialize(uint256 _denomination, uint32 _merkleTreeHeight) external { + function initialize(uint256 _denomination, uint32 _merkleTreeHeight) public virtual { // Part of ERC20TornadoCloneable.sol, denomination and levels is obviously // in all older code set once and never changed again, as such, we can use // these for `initialize`. The question comes, why is it set to 1 above?