Unnecessary struct in SingleNativeTokenExitV2

[code423n4]

Handle

kenzo

Vulnerability details

These are the structs in SingleNativeTokenExitV2: (Code ref)

struct ExitUnderlyingTrade {
        ExitUniswapV2SwapStruct[] swaps;
    }

    struct ExitUniswapV2SwapStruct {
        address exchange;
        address[] path;
    }
    struct ExitTokenStructV2 {
        address inputBasket;
        uint256 inputAmount;
        uint256 minAmount;
        uint256 deadline;
        uint16 referral;
        ExitUnderlyingTrade[] trades;
    }

ExitTokenStructV2 contains an array of ExitUnderlyingTrade , and ExitUnderlyingTrade is just an array of ExitUniswapV2SwapStruct. So ExitUnderlyingTrade is unnecessary.
For clarity, I suggest removing ExitUnderlyingTrade and just having an array of ExitUniswapV2SwapStruct in ExitTokenStructV2 .