We shouldn't mess with allocators that already are trying to collect resources.

olcCodeJam2023Entry/Scenario.cpp

@@ -112,7 +112,7 @@ void Scenario::RunAI(Resources&enemy_resources,std::vector<std::shared_ptr<Colle
 	//Randomly turn memory allocators into other units.
 	if(unitBuildTimer==0){
 		for(auto&u:units){
-			if(!u->IsFriendly()&&u->IsAllocator()){
+			if(!u->IsFriendly()&&u->IsAllocator()&&u->attachTarget.expired()){
 				std::array<UnitType,10>unitChoiceList={UnitType::LeftShifter,UnitType::LeftShifter,UnitType::RightShifter,UnitType::RightShifter,
 					UnitType::Corrupter,UnitType::MemorySwapper,UnitType::BitRestorer,UnitType::BitRestorer,UnitType::_Platform,UnitType::Corrupter};
 				std::array<std::vector<Memory>,10>unitResourceCostList={LeftShifter::resourceCost,LeftShifter::resourceCost,RightShifter::resourceCost,RightShifter::resourceCost,