Frame no longer needs to be mutable (refactoring artifact)

frame/contracts/src/exec.rs

@@ -653,18 +653,20 @@ where
 
 			// Additional work needs to be performed in case of an instantiation.
 			if output.is_success() && entry_point == ExportedFunction::Constructor {
-				let frame = self.top_frame_mut();
-				let account_id = frame.account_id.clone();
+				let frame = self.top_frame();
 
 				// It is not allowed to terminate a contract inside its constructor.
-				if let CachedContract::Terminated = frame.contract_info {
+				if matches!(frame.contract_info, CachedContract::Terminated) {
 					return Err(Error::<T>::TerminatedInConstructor.into())
 				}
 
 				// Deposit an instantiation event.
 				deposit_event::<T>(
 					vec![],
-					Event::Instantiated { deployer: self.caller().clone(), contract: account_id },
+					Event::Instantiated {
+						deployer: self.caller().clone(),
+						contract: frame.account_id.clone(),
+					},
 				);
 			}
 
@@ -743,7 +745,7 @@ where
 				return
 			}
 			if let CachedContract::Cached(contract) = &self.first_frame.contract_info {
-				<ContractInfoOf<T>>::insert(&self.first_frame.account_id, contract.clone());
+				<ContractInfoOf<T>>::insert(&self.first_frame.account_id, contract);
 			}
 			if let Some(counter) = self.account_counter {
 				<AccountCounter<T>>::set(counter);