ColumbusEngine

ColumbusEngine copied to clipboard

Hi, I tried to build the engine and open existing sample, but seems like scenes are missing some models & textures. Also there is a weird glitch in editor (probably...

MomoDeve

Improve exception safety with smart pointers

1

Would you like to [wrap any pointer data members](https://github.com/ColumbusUtrigas/ColumbusEngine/blob/d0c25a16c81ce17e81c51f33b861eb8541095691/Physics/PhysicsWorld.cpp#L7 "Update candidate: PhysicsWorld class") with the class template “[std::unique_ptr](https://en.wikipedia.org/wiki/Smart_pointer#unique_ptr "Description for the usage of smart pointers")”?

elfring

[Extra null pointer checks are not needed](https://isocpp.org/wiki/faq/freestore-mgmt#delete-handles-null "Do I need to check for null before delete p?") like in [the function “Sound::Free”](https://github.com/ColumbusUtrigas/ColumbusEngine/blob/99cac29f4dac86f01cca52760ac4b3e8f69b8d99/Common/Sound/Sound.cpp#L84 "Update candidate").

elfring