SameAssetUnderlyingsAbstractVault.removeVault currently calls SingleSlotMapper.removeValue with the removedValue parameter being the index of the _activeUnderlyingVaults array.
It would be more gas efficient if SameAssetUnderlyingsAbstractVault.removeVault called a new SingleSlotMapper.removeIndex function padding in the vaultIndex that is passed to the SameAssetUnderlyingsAbstractVault.removeVault function.
SameAssetUnderlyingsAbstractVault.removeVaultcurrently callsSingleSlotMapper.removeValuewith theremovedValueparameter being the index of the_activeUnderlyingVaultsarray.It would be more gas efficient if
SameAssetUnderlyingsAbstractVault.removeVaultcalled a newSingleSlotMapper.removeIndexfunction padding in thevaultIndexthat is passed to theSameAssetUnderlyingsAbstractVault.removeVaultfunction.