Applied the same treatment to GetWorldObjectsInDist Honestly no idea why the original idea insisted so much on using pointers. No more missing null checks or deletes. SmartAI's "ObjectList" is now a std::vector instead of a std::list because no where it was being used as an actual list. Original idea by DDuarte :P (cherry picked from commit a97439e8d63c0a78bda8a6ed4801f122385936e3)