feat(contracts-bedrock): add batch message support to L2toL2CrossDomainMessenger #12019
Conversation
|
This stack of pull requests is managed by Graphite. Learn more about stacking. Join @tremarkley and the rest of your teammates on |
![semgrep-app[bot]](https://avatars.githubusercontent.com/in/60555?s=60&v=4){kind=small}
| /// @notice Load a BatchableMessage from transient storage by index. | ||
| /// @param index The index of the message to load. | ||
| /// @return batchMessage The loaded BatchableMessage struct. | ||
| function _loadMessage(uint256 index) internal view returns (BatchableMessage memory batchMessage) { |
There was a problem hiding this comment.
Named return arguments to functions must be appended with an underscore (_)
|
|
||
| /// @notice Store batching status in transient storage. | ||
| /// @param value The batching status. | ||
| function _storeIsBatching(bool value) internal { |
There was a problem hiding this comment.
Inputs to functions must be prepended with an underscore (_)
|
|
||
| /// @notice Clear a specific BatchableMessage stored in transient storage by index. | ||
| /// @param index The index of the message to clear. | ||
| function _clearMessage(uint256 index) internal { |
There was a problem hiding this comment.
Inputs to functions must be prepended with an underscore (_)
| /// @notice Load a BatchableMessage from transient storage by index. | ||
| /// @param index The index of the message to load. | ||
| /// @return batchMessage The loaded BatchableMessage struct. | ||
| function _loadMessage(uint256 index) internal view returns (BatchableMessage memory batchMessage) { |
There was a problem hiding this comment.
Inputs to functions must be prepended with an underscore (_)
| /** | ||
| * @notice Clear all BatchableMessages stored in transient storage. | ||
| */ |
There was a problem hiding this comment.
Javadoc-style comments are not allowed. Use /// style doc comments instead.
|
Semgrep found 3
Do not use |
No description provided.