ETEngine

ETEngine copied to clipboard

An extra null pointer check is not needed in functions like the following.

• RawAsset<T>::UnloadInternal
• Scene::SetStarfield
• Singleton<T>::DestroyInstance

i use this

template <typename TElement>
inline void SafeDelete(TElement*& elementPointer)
{
    if (elementPointer)
    {
        delete elementPointer;
        elementPointer = nullptr;
    }
}

@codenamecpp: I find that your example for a function template contains an unnecessary null pointer check.