anon/instances
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

function visibility and inheritability

Browse Source 
Signed-off-by: T-Hax <>
This commit is contained in:
T-Hax 2023-06-13 20:02:45 +00:00
parent f5a302910a
commit eca31a997e
2 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
contracts/instances/ERC20TornadoCloneable.sol
View File

@ -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");

2
contracts/instances/ETHTornadoCloneable.sol
View File

@ -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?