feat: add named parameters to public mappings

MetisProtocol · Mar 5, 2024 · bbb77ff · bbb77ff
1 parent ca062df
commit bbb77ff
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/contracts/SequencerInfo.sol b/contracts/SequencerInfo.sol
@@ -8,7 +8,7 @@ contract SequencerInfo is OwnableUpgradeable, ISequencerInfo {
     uint256 public totalSequencers;
 
     // whitelist
-    mapping(address => bool) public whitelist;
+    mapping(address owner => bool yes) public whitelist;
 
     // sequencerId => sequencer
     mapping(uint256 seqId => Sequencer _seq) public sequencers;

diff --git a/contracts/SequencerSet.sol b/contracts/SequencerSet.sol
@@ -16,9 +16,9 @@ contract MetisSequencerSet is OwnableUpgradeable {
         uint256 endBlock;
     }
 
-    mapping(uint256 => Epoch) public epochs; // epoch number => epoch
+    mapping(uint256 number => Epoch epoch) public epochs; // epoch number => epoch
 
-    // current epoch id, starts from 0
+    // current epoch number, starts from 0
     uint256 internal currentEpochId;
 
     // event

diff --git a/contracts/test/TestBridge.sol b/contracts/test/TestBridge.sol
@@ -15,7 +15,7 @@ contract TestBridge is IL1ERC20Bridge {
         bytes _data
     );
 
-    mapping(address => uint256) public l2Balances;
+    mapping(address addr => uint256 balance) public l2Balances;
 
     function depositERC20ToByChainId(
         uint256,