Goal: Reduce the gas cost of functions based on usage frequency.

Possible changes Packing slugs in checkTransmitter of transmit

Accessed storage variable less: <div class="highlight highlight-source-solidity no

Contracts: gas optimisations about socket-dl HOT 5 CLOSED

socketdottech commented on August 11, 2024

Contracts: gas optimisations

from socket-dl.

Comments (5)

arthcp commented on August 11, 2024

Possible changes

Packing slugs in checkTransmitter of transmitManager
No need to emit capacitor address in PacketVerifiedAndSealed

from socket-dl.

arthcp commented on August 11, 2024

Mostly all nonReentrant modifiers can be removed

from socket-dl.

arthcp commented on August 11, 2024

Use location as storage instead of memory when reading structs from storage.

// bad
PlugConfig memory plugConfig = _plugConfigs[localPlug][remoteSlug];

// good
PlugConfig storage plugConfig = _plugConfigs[localPlug][remoteSlug];

from socket-dl.

ameeshaagrawal commented on August 11, 2024

Accessed storage variable less:

 uint256 packetCount = _sealedPackets++;

Removed redundant code:

imported SocketBase once with SocketDst
removed extra constructors

Packed _plugConfigs mapping:

// plug | remoteChainSlug => (siblingPlug, capacitor__, decapacitor__, inboundSwitchboard__, outboundSwitchboard__)
mapping(uint256 => PlugConfig) internal _plugConfigs;

Packed function params:

//bad
 function registerSwitchBoard(
        address switchBoardAddress_,
        uint256 siblingChainSlug_,
        uint256 capacitorType_
    ) external {

// good
 function registerSwitchBoard(
        address switchBoardAddress_,
        uint32 siblingChainSlug_,
        uint32 capacitorType_
    ) external {