[GEN][ZH] Simplify calls to MemoryPoolObject::deleteInstance (#870)

Author: xezon

Core/GameEngine/Source/Common/System/XferSave.cpp

@@ -91,7 +91,7 @@ XferSave::~XferSave( void )
 		{
 
 			next = m_blockStack->next;
-			MemoryPoolObject::deleteInstance(m_blockStack);
+			deleteInstance(m_blockStack);
 			m_blockStack = next;
 
 		}  // end while
@@ -247,7 +247,7 @@ void XferSave::endBlock( void )
 	fseek( m_fileFP, currentFilePos, SEEK_SET );
 
 	// delete the block data as it's all used up now
-	MemoryPoolObject::deleteInstance(top);
+	deleteInstance(top);
 
 }  // end endBlock
 

Generals/Code/GameEngine/Include/Common/GameMemory.h

@@ -767,6 +767,10 @@ class MemoryPoolObject
 	} 
 };
 
+inline void deleteInstance(MemoryPoolObject* mpo)
+{
+	MemoryPoolObject::deleteInstance(mpo);
+}
 
 
 // INLINING ///////////////////////////////////////////////////////////////////
@@ -906,7 +910,7 @@ class MemoryPoolObjectHolder
 	MemoryPoolObjectHolder(MemoryPoolObject *mpo = NULL) : m_mpo(mpo) { }
 	void hold(MemoryPoolObject *mpo) { DEBUG_ASSERTCRASH(!m_mpo, ("already holding")); m_mpo = mpo; }
 	void release() { m_mpo = NULL; }
-	~MemoryPoolObjectHolder() { MemoryPoolObject::deleteInstance(m_mpo); }
+	~MemoryPoolObjectHolder() { deleteInstance(m_mpo); }
 };
 
 

Generals/Code/GameEngine/Include/Common/GameMemoryNull.h

@@ -114,6 +114,11 @@ class MemoryPoolObject
 	}
 };
 
+inline void deleteInstance(MemoryPoolObject* mpo)
+{
+	MemoryPoolObject::deleteInstance(mpo);
+}
+
 
 /**
 	Initialize the memory manager. Construct a new MemoryPoolFactory and 

Generals/Code/GameEngine/Include/Common/Overridable.h

@@ -108,7 +108,7 @@ class Overridable : public MemoryPoolObject
 		{
 			if ( m_isOverride )
 			{
-				MemoryPoolObject::deleteInstance(this);
+				deleteInstance(this);
 				return NULL;
 			}
 			else if ( m_nextOverride )
@@ -123,7 +123,7 @@ class Overridable : public MemoryPoolObject
 __inline Overridable::~Overridable() 
 {
 	if (m_nextOverride) 
-		MemoryPoolObject::deleteInstance(m_nextOverride);
+		deleteInstance(m_nextOverride);
 }
 
 

Generals/Code/GameEngine/Include/Common/ThingTemplate.h

@@ -159,7 +159,7 @@ class AudioArray
 	{
 		for (Int i = 0; i < TTAUDIO_COUNT; ++i)
 			if (m_audio[i])
-				MemoryPoolObject::deleteInstance(m_audio[i]);
+				deleteInstance(m_audio[i]);
 	}
 
 	AudioArray(const AudioArray& that)

Generals/Code/GameEngine/Include/GameLogic/Object.h

@@ -386,7 +386,7 @@ class Object : public Thing, public Snapshot
 	Bool isInList(Object **pListHead) const;
 
 	// this is intended for use ONLY by GameLogic.
-	static void friend_deleteInstance(MemoryPoolObject* object) { MemoryPoolObject::deleteInstance(object); }
+	static void friend_deleteInstance(Object* object) { deleteInstance(object); }
 
 	/// cache the partition module (should be called only by PartitionData)
 	void friend_setPartitionData(PartitionData *pd) { m_partitionData = pd; }

Generals/Code/GameEngine/Include/GameLogic/ObjectIter.h

@@ -67,7 +67,7 @@ enum IterOrderType CPP_11(: Int)
 	{
 		// do something with other
 	}
-	MemoryPoolObject::deleteInstance(iter);								// you own it, so you must delete it
+	deleteInstance(iter);								// you own it, so you must delete it
 
 	note that the iterator is required to deal intelligently with deleted objects; 
 	in particular, next() will check if an obj has been killed and simply skip it.

Generals/Code/GameEngine/Source/Common/Audio/GameAudio.cpp

@@ -188,7 +188,7 @@ AudioManager::~AudioManager()
 	for (it = m_allAudioEventInfo.begin(); it != m_allAudioEventInfo.end(); ++it) {
 		AudioEventInfo *eventInfo = (*it).second;
 		if (eventInfo) {
-			MemoryPoolObject::deleteInstance(eventInfo);
+			deleteInstance(eventInfo);
 			eventInfo = NULL;
 		}
 	}
@@ -811,7 +811,7 @@ AudioRequest *AudioManager::allocateAudioRequest( Bool useAudioEvent )
 void AudioManager::releaseAudioRequest( AudioRequest *requestToRelease )
 {
 	if (requestToRelease) {
-		MemoryPoolObject::deleteInstance(requestToRelease);
+		deleteInstance(requestToRelease);
 	}
 }
 
@@ -894,7 +894,7 @@ void AudioManager::removeLevelSpecificAudioEventInfos(void)
 
     if ( it->second->isLevelSpecific() )
     {
-      MemoryPoolObject::deleteInstance(it->second);
+      deleteInstance(it->second);
       m_allAudioEventInfo.erase( it );
     }
 

Generals/Code/GameEngine/Source/Common/GameEngine.cpp

@@ -536,7 +536,7 @@ void GameEngine::reset( void )
 	if(background)
 	{
 		background->destroyWindows();
-		MemoryPoolObject::deleteInstance(background);
+		deleteInstance(background);
 		background = NULL;
 	}
 }

Generals/Code/GameEngine/Source/Common/GlobalData.cpp

@@ -1061,7 +1061,7 @@ GlobalData::~GlobalData( void )
 	DEBUG_ASSERTCRASH( TheWritableGlobalData->m_next == NULL, ("~GlobalData: theOriginal is not original\n") );
 
 	if (m_weaponBonusSet)
-		MemoryPoolObject::deleteInstance(m_weaponBonusSet);
+		deleteInstance(m_weaponBonusSet);
 
 	if( m_theOriginal == this )	{
 		m_theOriginal = NULL;