Sol-R

Sol-R copied to clipboard

Would you like to [wrap any pointer data members](https://github.com/cyrillefavreau/Sol-R/blob/983cc6953c23dc2b2e129379349b654eded40048/apps/scenes/animation/MetaballsScene.cpp#L54 "Update candidate: MetaballsScene 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

[An extra null pointer check is not needed in functions](https://isocpp.org/wiki/faq/freestore-mgmt#delete-handles-null "Do I need to check for null before delete p?") like the following. - [CPUKernel](https://github.com/cyrillefavreau/Sol-R/blob/983cc6953c23dc2b2e129379349b654eded40048/solr/engines/cpu/CPUKernel.cpp#L58 "Destructor for CPUKernel") - [GPUKernel::initBuffers](https://github.com/cyrillefavreau/Sol-R/blob/983cc6953c23dc2b2e129379349b654eded40048/solr/engines/GPUKernel.cpp#L345...

elfring