Functions not used internally could be marked external
Using proper visibility modifiers is a good practice to prevent unintended access to functions. Furthermore, marking functions as `external` instead of `public` can save gas.\\n```\\nFile: DepositVault.sol\\n\\n    function deposit(uint256 amount, address tokenAddress) public payable\\n\\n    function withdraw(uint256 amount, uint256 nonce, bytes memory signature, address payable recipient) public\\n\\n    function withdrawDeposit(uint256 depositIndex) public\\n```\\n
Consider change the visibility modifier to `external` for the functions that are not used internally.
```\\nFile: DepositVault.sol\\n\\n    function deposit(uint256 amount, address tokenAddress) public payable\\n\\n    function withdraw(uint256 amount, uint256 nonce, bytes memory signature, address payable recipient) public\\n\\n    function withdrawDeposit(uint256 depositIndex) public\\n```\\n