Redundant `reserveRecoverGas` param in `_swapAndCompleteBridgeTokens` function

[sujithsomraaj]

Context: ReceiverStargateV2.sol#L147

Description: The function _swapAndCompleteBridgeTokens executes a destination chain swap post-stargate bridging. This function is invoked through the lzCompose call from the Layerzero endpoint.

This function has a parameter called reserveRecoverGas, which is always passed as true in all possible paths. Hence, the function could be simplified by removing the parameter to optimize gas.

Recommendation: Consider removing the unused parameter from the function.

    function _swapAndCompleteBridgeTokens(
        bytes32 _transactionId,
        LibSwap.SwapData[] memory _swapData,
        address assetId,
        address payable receiver,
        uint256 amount,
-        bool reserveRecoverGas
    ) private {
-    uint256 _recoverGas = reserveRecoverGas ? recoverGas : 0;
+    uint256 _recoverGas = recoverGas;

LI.FI:

Researcher:

[0xDEnYO]

Great suggestion. Removed it.