Changeset 20855

Timestamp:

09/25/07 10:51:44 (1 year ago)

Author:

patrick

Message:

Take some steps to centralize the creation of the string form of an OpenAL
error. This really isn't much better than what was already there, but I feel
that it is going in the right direction.

Files:

• juggler/branches/2.2/modules/sonix/plugins/OpenAL/OpenALSoundImplementation.cpp (modified) (4 diffs)

juggler/branches/2.2/modules/sonix/plugins/OpenAL/OpenALSoundImplementation.cpp

@@ -116 +116 @@
-/////////////////////////
-
+namespace
+{
+
+std::string getAlErrorString(const ALenum errCode)
+{
+   std::string msg;
+
+   switch ( errCode )
+   {
+      case AL_NO_ERROR:
+         msg = "AL_NO_ERROR";
+         break;
+      case AL_INVALID_NAME:
+         msg = "AL_INVALID_NAME";
+         break;
+      case AL_INVALID_ENUM:
+         msg = "AL_INVALID_ENUM";
+         break;
+      case AL_INVALID_VALUE:
+         msg = "AL_INVALID_VALUE";
+         break;
+      case AL_INVALID_OPERATION:
+         msg = "AL_INVALID_OPERATION";
+         break;
+      case AL_OUT_OF_MEMORY:
+         msg = "AL_OUT_OF_MEMORY";
+         break;
+   }
+
+   return msg;
+}
+
+}
+
-namespace snx
-{
@@ -697 +731 @@
-         if (err != AL_NO_ERROR)
-         {
-CONFI
+WARNIN
-               << clrOutNORM(clrYELLOW, "ERROR:")
-               << "Could not gen a buffer [err=" << err << "]\n"
-               << vprDEBUG_FLUSH;
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
-
+
+
-         }
-
@@ -853 +871 @@
-   if (mBindLookup.count( alias ) > 0)
-   {
-int
+ALenum
-      alDeleteSources( 1, &mBindLookup[alias].source );
-      err = alGetError();
-      if (err != AL_NO_ERROR)
-      {
-
-
+
+
+
+
+
-      }
-      alDeleteBuffers( 1, &mBindLookup[alias].buffer );
@@ -865 +886 @@
-      if (err != AL_NO_ERROR)
-      {
-
-
+
+
+
+
+
-      }
-      mBindLookup.erase( alias );