Skip to content

Commit

Permalink
contracts-bedrock: remove dependencySet from initialize in SystemConfig
Browse files Browse the repository at this point in the history
  • Loading branch information
0xfuturistic committed May 10, 2024
1 parent 25be07c commit 87d2d34
Showing 1 changed file with 2 additions and 9 deletions.
11 changes: 2 additions & 9 deletions packages/contracts-bedrock/src/L1/SystemConfig.sol
Original file line number Diff line number Diff line change
Expand Up @@ -154,8 +154,7 @@ contract SystemConfig is OwnableUpgradeable, ISemver, IGasToken {
optimismPortal: address(0),
optimismMintableERC20Factory: address(0),
gasPayingToken: address(0)
}),
_dependencySet: new uint256[](0)
})
});
}

Expand All @@ -171,7 +170,6 @@ contract SystemConfig is OwnableUpgradeable, ISemver, IGasToken {
/// @param _batchInbox Batch inbox address. An identifier for the op-node to find
/// canonical data.
/// @param _addresses Set of L1 contract addresses. These should be the proxies.
/// @param _dependencySet Initial interop dependency set.
function initialize(
address _owner,
uint256 _overhead,
Expand All @@ -181,8 +179,7 @@ contract SystemConfig is OwnableUpgradeable, ISemver, IGasToken {
address _unsafeBlockSigner,
ResourceMetering.ResourceConfig memory _config,
address _batchInbox,
SystemConfig.Addresses memory _addresses,
uint256 _dependencySet
SystemConfig.Addresses memory _addresses
)
public
initializer
Expand All @@ -209,10 +206,6 @@ contract SystemConfig is OwnableUpgradeable, ISemver, IGasToken {

_setResourceConfig(_config);
require(_gasLimit >= minimumGasLimit(), "SystemConfig: gas limit too low");

for (uint256 i = 0; i < _dependencySet; i++) {
_addDependency(i);
}
}

/// @notice Returns the minimum L2 gas limit that can be safely set for the system to
Expand Down

0 comments on commit 87d2d34

Please sign in to comment.