Commit 7d7d9ba8 authored by Kelvin Fichter's avatar Kelvin Fichter

fix(ctb): move inits under constructors

Standardizes the location of initializers to be under constructors.
parent cc738b3d
---
'@eth-optimism/contracts-bedrock': patch
---
Moves initializers underneath constructors always
...@@ -99,6 +99,14 @@ contract OptimismPortal is Initializable, ResourceMetering, Semver { ...@@ -99,6 +99,14 @@ contract OptimismPortal is Initializable, ResourceMetering, Semver {
initialize(); initialize();
} }
/**
* @notice Initializer;
*/
function initialize() public initializer {
l2Sender = DEFAULT_L2_SENDER;
__ResourceMetering_init();
}
/** /**
* @notice Accepts value so that users can send ETH directly to this contract and have the * @notice Accepts value so that users can send ETH directly to this contract and have the
* funds be deposited to their address on L2. This is intended as a convenience * funds be deposited to their address on L2. This is intended as a convenience
...@@ -214,14 +222,6 @@ contract OptimismPortal is Initializable, ResourceMetering, Semver { ...@@ -214,14 +222,6 @@ contract OptimismPortal is Initializable, ResourceMetering, Semver {
return _isOutputFinalized(proposal); return _isOutputFinalized(proposal);
} }
/**
* @notice Initializer;
*/
function initialize() public initializer {
l2Sender = DEFAULT_L2_SENDER;
__ResourceMetering_init();
}
/** /**
* @notice Accepts deposits of ETH and data, and emits a TransactionDeposited event for use in * @notice Accepts deposits of ETH and data, and emits a TransactionDeposited event for use in
* deriving deposit transactions. Note that if a deposit is made by a contract, its * deriving deposit transactions. Note that if a deposit is made by a contract, its
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment