Changeset 20871

Timestamp:

09/26/07 14:49:39 (1 year ago)

Author:

patrick

Message:

Added snx::sonix::shutdownAPI() so that user-level code has an opportunity
to control when the sound API is shut down. Previously, this only happend in
the snx::sonix destructor, a function that is only invoked when the snx::sonix
singleton is destroyed. Technically, user-level code could have used
snx::sonix::changeAPI() to switch to the "Stub" sound implementation, but that
does not exactly have the desired outcome in my opinion.

This change allows for the VR Juggler Sound Manager to shut down Sonix so that
crash-on-exit problems in the Sonix plug-ins can be fixed. The issues with
Audiere and OpenAL seemed to be tied to threading. More specifically, the
sound APIs are created and initialized in the kernel control loop thread, but
they were destroyed in the primordial thread at the time of application exit.
Now, the Sound Manager can shut Sonix down from within the kernel control loop
thread.

Bumped the version to 1.3.6.

Files:

• juggler/trunk/modules/sonix/ChangeLog (modified) (1 diff)
• juggler/trunk/modules/sonix/VERSION (modified) (1 diff)
• juggler/trunk/modules/sonix/snx/sonix.cpp (modified) (2 diffs)
• juggler/trunk/modules/sonix/snx/sonix.h (modified) (1 diff)

juggler/trunk/modules/sonix/ChangeLog

@@ -1 +1 @@
-DATE        AUTHOR      CHANGE
------------ ----------- -------------------------------------------------------
+Sep-26-2007 patrick     Added snx::sonix::shutdownAPI() so that user-level code
+                        can control when the sound API is closed rather than
+                        having it happen only in the snx::sonix destructor.
+                        NEW VERSION: 1.3.6
-Sep-25-2007 patrick     Changed method signatures to remove passing of
-                        fundamental types by const reference.

juggler/trunk/modules/sonix/VERSION

@@ +1 @@
+1.3.6-0 @09/26/2007 19:50:00 UTC@
-1.3.5-0 @09/25/2007 21:35:00 UTC@
-1.3.4-0 @09/25/2007 21:30:00 UTC@

juggler/trunk/modules/sonix/snx/sonix.cpp

@@ -58 +58 @@
-sonix::~sonix()
-{
-
-
-
-
-
-
-
-
-
-
+
+
+
-   }
-}
@@ -230 +223 @@
-}
-
+// Destructor.
+void sonix::shutdownAPI()
+{
+   // release the implementation
+   if ( mImplementation != NULL )
+   {
+      // unload all sound data
+      mImplementation->unbindAll();
+
+      // shutdown old api if exists
+      mImplementation->shutdownAPI();
+      delete mImplementation;
+      mImplementation = NULL;
+   }
+}
+
-void sonix::configure(const snx::SoundAPIInfo& sai)
-{

juggler/trunk/modules/sonix/snx/sonix.h

@@ -243 +243 @@
-
-   /**
+    * If there is a sound API currently active, its sounds are unbound, and
+    * it is told to shut itself down.
+    *
+    * @post \c mImplementation is NULL.
+    *
+    * @since 1.3.6
+    */
+   void shutdownAPI();
+
+   /**
-    * Configures/reconfigures the sound API global settings.
-    *