delete controllers[] in removeVault?

[gpersoon]

The function removeVault() deletes info about a vault.

Perhaps it should also delete controllers[]?

Example code:

delete controllers[_vault];

metavault/contracts/v3/Manager.sol

Lines 430 to 445 in 6dcff91

	* @notice Removes a token from being able to be deposited for a given vault
	* @param _vault The address of the vault
	*/
	function removeVault(
	address _vault
	)
	external
	override
	notHalted
	onlyStrategist
	{
	require(tokens[_vault] != address(0), "!_vault");
	delete tokens[_vault];
	delete allowedVaults[_vault];
	emit VaultRemoved(_vault);
	}

metavault/contracts/v3/Manager.sol

Lines 452 to 464 in 6dcff91

	function setController(
	address _vault,
	address _controller
	)
	external
	notHalted
	onlyStrategist
	{
	require(allowedVaults[_vault], "!_vault");
	require(allowedControllers[_controller], "!_controller");
	controllers[_vault] = _controller;
	emit SetController(_vault, _controller);
	}