Changeset 4960 in orxonox.OLD

Timestamp:

Jul 27, 2005, 6:27:03 PM (19 years ago)

Author:

bensch

Message:

orxonox/trunk: the SoundEngine now gets the SoundBuffer- and SoundSource - Lists directly from the ClassList, and therefor we have much less overhead in the process of alocating deleting them (/2)

Location:

orxonox/trunk/src/lib/sound

Files:

• sound_engine.cc (modified) (13 diffs)
• sound_engine.h (modified) (3 diffs)

orxonox/trunk/src/lib/sound/sound_engine.cc

@@ -22 +22 @@
 
 //#include <AL/alc.h> // maybe later
+#include "class_list.h"
 
 #include "p_node.h"
@@ -81 +82 @@
 SoundBuffer::~SoundBuffer()
 {
-  SoundEngine::getInstance()->removeBuffer(this);
+//  SoundEngine::getInstance()->removeBuffer(this);
   alDeleteBuffers(1, &this->bufferID);
 }
@@ -118 +119 @@
 SoundSource::~SoundSource()
 {
-  SoundEngine::getInstance()->removeSource(this);
+  //SoundEngine::getInstance()->removeSource(this);
   alDeleteSources(1, &this->sourceID);
 }
@@ -192 +193 @@
 
   this->listener = NULL;
-  this->bufferList = new tList<SoundBuffer>;
-  this->sourceList = new tList<SoundSource>;
+  this->bufferList = NULL;
+  this->sourceList = NULL;
 }
 
@@ -210 +211 @@
 
   // deleting all the SoundSources
-  tIterator<SoundSource>* sourceIterator = this->sourceList->getIterator();
-  SoundSource* enumSource = sourceIterator->nextElement();
-  while (enumSource)
-    {
-      delete enumSource;
-      enumSource = sourceIterator->nextElement();
-    }
-  delete sourceIterator;
+  if(this->sourceList != NULL)
+  {
+    tIterator<BaseObject>* sourceIterator = this->sourceList->getIterator();
+    SoundSource* enumSource = (SoundSource*)sourceIterator->nextElement();
+    while (enumSource)
+    {
+        delete enumSource;
+        enumSource = (SoundSource*)sourceIterator->nextElement();
+    }
+    delete sourceIterator;
+  }
 
   // deleting all the SoundBuffers
-  tIterator<SoundBuffer>* bufferIterator = this->bufferList->getIterator();
-  SoundBuffer* enumBuffer = bufferIterator->nextElement();
-  while (enumBuffer)
+  if (this->bufferList != NULL)
+  {
+    tIterator<BaseObject>* bufferIterator = this->bufferList->getIterator();
+    SoundBuffer* enumBuffer = (SoundBuffer*)bufferIterator->nextElement();
+    while (enumBuffer)
     {
       ResourceManager::getInstance()->unload(enumBuffer);
-      enumBuffer = bufferIterator->nextElement();
-    }
-  delete bufferIterator;
-
+      enumBuffer = (SoundBuffer*)bufferIterator->nextElement();
+    }
+    delete bufferIterator;
+  }
   // removing openAL from AudioResource
   alutExit();
@@ -273 +279 @@
 void SoundEngine::addBuffer(SoundBuffer* buffer)
 {
-  this->bufferList->add(buffer);
+  if (unlikely(this->bufferList == NULL))
+    this->bufferList = ClassList::getList(CL_SOUND_BUFFER);
 }
 
@@ -283 +290 @@
 {
   // look if there are any sources that have the buffer still loaded
-  tIterator<SoundSource>* sourceIterator = this->sourceList->getIterator();
-  SoundSource* enumSource = sourceIterator->nextElement();
+  tIterator<BaseObject>* sourceIterator = this->sourceList->getIterator();
+  SoundSource* enumSource = (SoundSource*)sourceIterator->nextElement();
   while (enumSource)
     {
       if (buffer == enumSource->getBuffer())
         delete enumSource;
-      enumSource = sourceIterator->nextElement();
+      enumSource = (SoundSource*)sourceIterator->nextElement();
     }
   delete sourceIterator;
-
-  // remove the Buffer
-  this->bufferList->remove(buffer);
 }
 
@@ -303 +307 @@
 void SoundEngine::addSource(SoundSource* source)
 {
-  this->sourceList->add(source);
-}
-
-/**
- *  removes a SoundSource from the sourceList of the SoundEngine
- * @param source The source to delete from the SoundEngine
-*/
-void SoundEngine::removeSource(SoundSource* source)
-{
-  this->sourceList->remove(source);
-}
-
+  this->sourceList = ClassList::getList(CL_SOUND_SOURCE);
+}
 
 /**
@@ -341 +335 @@
 
   // updating all the Sources positions
-  tIterator<SoundSource>* iterator = this->sourceList->getIterator();
-  SoundSource* enumSource = iterator->nextElement();
-  while (enumSource)
+  if (likely(this->sourceList != NULL))
+  {
+    tIterator<BaseObject>* iterator = this->sourceList->getIterator();
+    SoundSource* enumSource = (SoundSource*)iterator->nextElement();
+    while (enumSource)
     {
       if (likely(enumSource->getNode()!=NULL))
@@ -356 +352 @@
                    enumSource->getNode()->getVelocity().z);
       }
-      enumSource = iterator->nextElement();
-    }
-  delete iterator;
+      enumSource = (SoundSource*)iterator->nextElement();
+    }
+    delete iterator;
+  }
 }
 
@@ -366 +363 @@
 void SoundEngine::flushUnusedBuffers()
 {
-  tIterator<SoundBuffer>* bufferIterator = this->bufferList->getIterator();
-  SoundBuffer* enumBuffer = bufferIterator->nextElement();
-  while (enumBuffer)
-    {
-      tIterator<SoundSource>* sourceIterator = this->sourceList->getIterator();
-      SoundSource* enumSource = sourceIterator->nextElement();
+  if(this->sourceList && this->bufferList)
+  {
+    tIterator<BaseObject>* bufferIterator = this->bufferList->getIterator();
+    SoundBuffer* enumBuffer = (SoundBuffer*)bufferIterator->nextElement();
+    while (enumBuffer)
+    {
+      tIterator<BaseObject>* sourceIterator = this->sourceList->getIterator();
+      SoundSource* enumSource = (SoundSource*)sourceIterator->nextElement();
       while (enumSource)
-        {
-          if (enumBuffer == enumSource->getBuffer())
-            break;
-          enumSource = sourceIterator->nextElement();
-        }
+      {
+        if (enumBuffer == enumSource->getBuffer())
+          break;
+        enumSource = (SoundSource*)sourceIterator->nextElement();
+      }
       delete sourceIterator;
       if (enumSource == NULL)
         ResourceManager::getInstance()->unload(enumBuffer);
-      enumBuffer = bufferIterator->nextElement();
-    }
-  delete bufferIterator;
+      enumBuffer = (SoundBuffer*)bufferIterator->nextElement();
+    }
+    delete bufferIterator;
+  }
 }
 
@@ -391 +391 @@
 void SoundEngine::flushAllBuffers()
 {
-  tIterator<SoundBuffer>* bufferIterator = this->bufferList->getIterator();
-  SoundBuffer* enumBuffer = bufferIterator->nextElement();
-  while (enumBuffer)
+  if (this->bufferList)
+  {
+    tIterator<BaseObject>* bufferIterator = this->bufferList->getIterator();
+    SoundBuffer* enumBuffer = (SoundBuffer*)bufferIterator->nextElement();
+    while (enumBuffer)
     {
       ResourceManager::getInstance()->unload(enumBuffer, RP_LEVEL);
-      enumBuffer = bufferIterator->nextElement();
-    }
-  delete bufferIterator;
+      enumBuffer = (SoundBuffer*)bufferIterator->nextElement();
+    }
+    delete bufferIterator;
+  }
 }
 
@@ -406 +409 @@
 void SoundEngine::flushAllSources()
 {
-  tIterator<SoundSource>* Iterator = this->sourceList->getIterator();
-  SoundSource* enumSource = Iterator->nextElement();
-  while (enumSource)
+  if (this->sourceList)
   {
-    delete enumSource;
-    enumSource = Iterator->nextElement();
+    tIterator<BaseObject>* Iterator = this->sourceList->getIterator();
+    SoundSource* enumSource = (SoundSource*)Iterator->nextElement();
+    while (enumSource)
+    {
+      delete enumSource;
+      enumSource = (SoundSource*)Iterator->nextElement();
+    }
+    delete Iterator;
   }
-  delete Iterator;
-}
-
+}
 
 /**

orxonox/trunk/src/lib/sound/sound_engine.h

@@ -9 +9 @@
 #include "base_object.h"
 #include "alincl.h"
-
 
 #define SOUND_DOPPLER_FACTOR       0.001          //!< A factor for the audible doppler effect
@@ -85 +84 @@
     void removeBuffer(SoundBuffer* buffer);
     void addSource(SoundSource* source);
-    void removeSource(SoundSource* source);
 
     void update();
@@ -109 +107 @@
 
     PNode*                   listener;                 //!< The listener of the Scene
-    tList<SoundBuffer>*      bufferList;               //!< A list of buffers
-    tList<SoundSource>*      sourceList;               //!< A list for all the sources in the scene.
+    tList<BaseObject>*       bufferList;               //!< A list of buffers
+    tList<BaseObject>*       sourceList;               //!< A list for all the sources in the scene.
 
 };