Skip to content

Commit

Permalink
fix: use calldata for unmodified parameter
Browse files Browse the repository at this point in the history
  • Loading branch information
adu-web3 committed Aug 27, 2024
1 parent e243963 commit d2bbefb
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion src/core/ClientChainGateway.sol
Original file line number Diff line number Diff line change
Expand Up @@ -115,7 +115,7 @@ contract ClientChainGateway is
}

/// @inheritdoc IClientChainGateway
function quote(bytes memory _message) public view returns (uint256 nativeFee) {
function quote(bytes calldata _message) public view returns (uint256 nativeFee) {
bytes memory options = OptionsBuilder.newOptions().addExecutorLzReceiveOption(
DESTINATION_GAS_LIMIT, DESTINATION_MSG_VALUE
).addExecutorOrderedExecutionOption();
Expand Down
2 changes: 1 addition & 1 deletion src/core/CustomProxyAdmin.sol
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ contract CustomProxyAdmin is Initializable, ProxyAdmin {
/// @param implementation The address of the new implementation contract.
/// @param data The data to be passed to the new implementation contract.
/// @dev This function can only be called by the proxy to upgrade itself, exactly once.
function changeImplementation(address proxy, address implementation, bytes memory data) public virtual {
function changeImplementation(address proxy, address implementation, bytes calldata data) public virtual {
if (msg.sender != bootstrapper) {
revert Errors.CustomProxyAdminOnlyCalledFromBootstrapper();
}
Expand Down
2 changes: 1 addition & 1 deletion src/core/ExocoreGateway.sol
Original file line number Diff line number Diff line change
Expand Up @@ -606,7 +606,7 @@ contract ExocoreGateway is
}

/// @inheritdoc IExocoreGateway
function quote(uint32 srcChainid, bytes memory _message) public view returns (uint256 nativeFee) {
function quote(uint32 srcChainid, bytes calldata _message) public view returns (uint256 nativeFee) {
bytes memory options = OptionsBuilder.newOptions().addExecutorLzReceiveOption(
DESTINATION_GAS_LIMIT, DESTINATION_MSG_VALUE
).addExecutorOrderedExecutionOption();
Expand Down

0 comments on commit d2bbefb

Please sign in to comment.