oh no, all the easy smart-pointer conversions are done :(

gotta bite the bullet and get into the entity management

CLevelScene::~CLevelScene()
{
	// Deletes all the non-entity owned boundings, since entities delete their own boundings.
	for (auto bounding : m_Boundings)
	{
		if (bounding && !bounding->GetEntity()) delete bounding;
	}
}

It's this stuff that is hella scary

really makes me scratch my head thinking "how the fuck am I gonna do this"

Okay, so my Level class owns all entities in the level

Currently, if an entity wants to spawn another entity, it can simply new one, add it to the level, and keep a pointer to the entity it allocated.

Now, I want to replace that allocation with std::unique_ptr

But if I pass the entity that was std::make_uniqued to the level, it will of course std::move it into a std::vector, and the variable that was initialized in the caller's code won't be valid anymore.

Is there a common way to fix this? Do I make Level::AddEntity return a raw pointer to the entity that is moved into the container in Level?

m_Child = new CEntity();
level->AddEntity(child);
m_Child->DoStuff();

VS

m_Child = std::make_unique<CEntity>();
level->AddEntity(std::move(m_Child));
m_Child->DoStuff(); // WHOOOPS, RUNTIME ERROR (hopefully)