johnbartholomew/pioneer — comment on commit dbcafefd4cec24e1211d783b2b0f7813f2d8beb0
File: src/Game.cpp line 82
By robn:

> I haven't looked at Space closely enough to guess whether it's right to expose UpdateBodies directly though.

It's not really the right thing to do.

> Another option (I think) is for Player (or perhaps somewhere further up the class hierarchy) to forcefully remove itself from Space on destruction.

The trouble there is that it requires the Player to know which Space its part of. So far I've managed to avoid that.

> Changing the declaration order of m_space and m_player ... [is] pretty horrible...

Agreed. I'm not going there.

I have half an idea brewing, I'll try it this morning if I get time.