SKALE contest - d4rk's results

Initializing i with its default value isn’t needed and costs extra gas. MessageProxy.sol::515 - for (uint256 i = 0; i < messages.length; i++) {

Reading the array length for each iteration spends more gas in comparison to caching the len outside of the loop: MessageProxy.sol::515 - for (uint256 i = 0; i < messages.length; i++) {

Changing "constant" to "immutable" will result in computing the hash once during deployment thus saving gas rather than with the constant, where the computation of keccak256 operation occurs each time the following roles are used: MessageProxy.sol::50 - bytes32 public constant MAINNET_HASH = keccak256(abi.encodePacked("Mainnet")); MessageProxy.sol::51 - bytes32 public constant CHAIN_CONNECTOR_ROLE = keccak256("CHAIN_CONNECTOR_ROLE"); MessageProxy.sol::52 - bytes32 public constant EXTRA_CONTRACT_REGISTRAR_ROLE = keccak256("EXTRA_CONTRACT_REGISTRAR_ROLE"); MessageProxy.sol::53 - bytes32 public constant CONSTANT_SETTER_ROLE = keccak256("CONSTANT_SETTER_ROLE");