diff src/cdrom/macosx/CDPlayer.c @ 1487:dc6b59e925a2

Cleaning up warnings on MacOS X
author Sam Lantinga <slouken@libsdl.org>
date Thu, 09 Mar 2006 06:33:21 +0000
parents d910939febfa
children 782fd950bd46 c121d94672cb a1b03ba2fcd0
line wrap: on
line diff
--- a/src/cdrom/macosx/CDPlayer.c	Wed Mar 08 18:30:12 2006 +0000
+++ b/src/cdrom/macosx/CDPlayer.c	Thu Mar 09 06:33:21 2006 +0000
@@ -25,16 +25,16 @@
 #include "AudioFilePlayer.h"
 #include "SDLOSXCAGuard.h"
 
-// we're exporting these functions into C land for SDL_syscdrom.c
-//extern "C" {
+/* we're exporting these functions into C land for SDL_syscdrom.c */
+/*extern "C" {*/
 
-//ΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡ
-//  Constants
-//ΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡ
+/*///////////////////////////////////////////////////////////////////////////
+    Constants
+  //////////////////////////////////////////////////////////////////////////*/
 
-#define kAudioCDFilesystemID   (UInt16)(('J' << 8) | 'H') // 'JH'; this avoids compiler warning
+#define kAudioCDFilesystemID   (UInt16)(('J' << 8) | 'H') /* 'JH'; this avoids compiler warning */
 
-// XML PList keys
+/* XML PList keys */
 #define kRawTOCDataString           "Format 0x02 TOC Data"
 #define kSessionsString             "Sessions"
 #define kSessionTypeString          "Session Type"
@@ -47,9 +47,9 @@
 #define kSessionNumberKeyString         "Session Number"
 #define kStartBlockKeyString            "Start Block"   
     
-//ΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡ
-//  Globals
-//ΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡ
+/*///////////////////////////////////////////////////////////////////////////
+    Globals
+  //////////////////////////////////////////////////////////////////////////*/
 
 #pragma mark -- Globals --
 
@@ -61,9 +61,9 @@
 static SDL_sem         *callbackSem;
 static SDL_CD*          theCDROM;
 
-//ΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡ
-//  Prototypes
-//ΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡ
+/*///////////////////////////////////////////////////////////////////////////
+    Prototypes
+  //////////////////////////////////////////////////////////////////////////*/
 
 #pragma mark -- Prototypes --
 
@@ -89,10 +89,6 @@
     SDL_mutexV(apiMutex);
 }
 
-//ΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡ
-//  DetectAudioCDVolumes
-//ΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡ
-
 int DetectAudioCDVolumes(FSVolumeRefNum *volumes, int numVolumes)
 {
     int volumeIndex;
@@ -116,7 +112,7 @@
          
         if (result == noErr)
         {
-            if (volumeInfo.filesystemID == kAudioCDFilesystemID) // It's an audio CD
+            if (volumeInfo.filesystemID == kAudioCDFilesystemID) /* It's an audio CD */
             {
                 if (volumes != NULL && cdVolumeCount < numVolumes)
                     volumes[cdVolumeCount] = actualVolume;
@@ -126,18 +122,14 @@
         }
         else 
         {
-            // I'm commenting this out because it seems to be harmless
-            //SDL_SetError ("DetectAudioCDVolumes: FSGetVolumeInfo returned %d", result);
+            /* I'm commenting this out because it seems to be harmless */
+            /*SDL_SetError ("DetectAudioCDVolumes: FSGetVolumeInfo returned %d", result);*/
         }
     }
         
     return cdVolumeCount;
 }
 
-//ΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡ
-//  ReadTOCData
-//ΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡ
-
 int ReadTOCData (FSVolumeRefNum theVolume, SDL_CD *theCD)
 {
     HFSUniStr255      dataForkName;
@@ -154,7 +146,7 @@
     
     const char* error = "Unspecified Error";
     
-    // get stuff from .TOC.plist                                                   
+    /* get stuff from .TOC.plist */
     fsRefPB.ioCompletion = NULL;
     fsRefPB.ioNamePtr = "\p.TOC.plist";
     fsRefPB.ioVRefNum = theVolume;
@@ -167,7 +159,7 @@
         goto bail;
     }
     
-    // Load and parse the TOC XML data
+    /* Load and parse the TOC XML data */
 
     theErr = FSGetDataForkName (&dataForkName);
     if (theErr != noErr) {
@@ -187,7 +179,7 @@
         goto bail;
     }
     
-    // Allocate some memory for the XML data
+    /* Allocate some memory for the XML data */
     forkData = NewPtr (forkSize);
     if(forkData == NULL) {
         error = "NewPtr";
@@ -215,9 +207,9 @@
         goto bail;
     }
 
-    // Now we got the Property List in memory. Parse it.
+    /* Now we got the Property List in memory. Parse it. */
     
-    // First, make sure the root item is a CFDictionary. If not, release and bail.
+    /* First, make sure the root item is a CFDictionary. If not, release and bail. */
     if(CFGetTypeID(propertyListRef)== CFDictionaryGetTypeID())
     {
         CFDictionaryRef dictRef = (CFDictionaryRef)propertyListRef;
@@ -227,19 +219,19 @@
         CFIndex     numSessions;
         CFIndex     index;
         
-        // This is how we get the Raw TOC Data
+        /* This is how we get the Raw TOC Data */
         theRawTOCDataRef = (CFDataRef)CFDictionaryGetValue (dictRef, CFSTR(kRawTOCDataString));
         
-        // Get the session array info.
+        /* Get the session array info. */
         theSessionArrayRef = (CFArrayRef)CFDictionaryGetValue (dictRef, CFSTR(kSessionsString));
         
-        // Find out how many sessions there are.
+        /* Find out how many sessions there are. */
         numSessions = CFArrayGetCount (theSessionArrayRef);
         
-        // Initialize the total number of tracks to 0
+        /* Initialize the total number of tracks to 0 */
         theCD->numtracks = 0;
         
-        // Iterate over all sessions, collecting the track data
+        /* Iterate over all sessions, collecting the track data */
         for(index = 0; index < numSessions; index++)
         {
             CFDictionaryRef theSessionDict;
@@ -272,7 +264,7 @@
                 startBlock    = (CFNumberRef)  CFDictionaryGetValue (theTrackDict, CFSTR(kStartBlockKeyString));
                 isDataTrack   = (CFBooleanRef) CFDictionaryGetValue (theTrackDict, CFSTR(kDataKeyString));
                                                         
-                // Fill in the SDL_CD struct
+                /* Fill in the SDL_CD struct */
                 int idx = theCD->numtracks++;
 
                 CFNumberGetValue (trackNumber, kCFNumberSInt32Type, &value);
@@ -283,19 +275,19 @@
 
                 theCD->track[idx].type = (isDataTrack == kCFBooleanTrue) ? SDL_DATA_TRACK : SDL_AUDIO_TRACK;
 
-                // Since the track lengths are not stored in .TOC.plist we compute them.
+                /* Since the track lengths are not stored in .TOC.plist we compute them. */
                 if (trackIndex > 0) {
                     theCD->track[idx-1].length = theCD->track[idx].offset - theCD->track[idx-1].offset;
                 }
             }
             
-            // Compute the length of the last track
+            /* Compute the length of the last track */
             CFNumberGetValue (leadoutBlock, kCFNumberSInt32Type, &value);
             
             theCD->track[theCD->numtracks-1].length = 
                 value - theCD->track[theCD->numtracks-1].offset;
 
-            // Set offset to leadout track
+            /* Set offset to leadout track */
             theCD->track[theCD->numtracks].offset = value;
         }
     
@@ -320,10 +312,6 @@
     return theErr;
 }
 
-//ΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡ
-//  ListTrackFiles
-//ΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡ
-
 int ListTrackFiles (FSVolumeRefNum theVolume, FSRef *trackFiles, int numTracks)
 {
     OSStatus        result = -1;
@@ -357,11 +345,11 @@
                 CFStringRef  name;
                 name = CFStringCreateWithCharacters (NULL, nameStr.unicode, nameStr.length);
                 
-                // Look for .aiff extension
+                /* Look for .aiff extension */
                 if (CFStringHasSuffix (name, CFSTR(".aiff")) ||
                     CFStringHasSuffix (name, CFSTR(".cdda"))) {
                     
-                    // Extract the track id from the filename
+                    /* Extract the track id from the filename */
                     int trackID = 0, i = 0;
                     while (i < nameStr.length && !isdigit(nameStr.unicode[i])) {
                         ++i;
@@ -376,7 +364,7 @@
                     CFStringGetCStringPtr (name, CFStringGetSystemEncoding()));
                     #endif
                     
-                    // Track ID's start at 1, but we want to start at 0
+                    /* Track ID's start at 1, but we want to start at 0 */
                     trackID--;
                     
                     assert(0 <= trackID && trackID <= SDL_MAX_TRACKS);
@@ -393,10 +381,6 @@
     return 0;
 }
 
-//ΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡ
-//  LoadFile
-//ΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡ
-
 int LoadFile (const FSRef *ref, int startFrame, int stopFrame)
 {
     int error = -1;
@@ -404,7 +388,7 @@
     if (CheckInit () < 0)
         goto bail;
     
-    // release any currently playing file
+    /* release any currently playing file */
     if (ReleaseFile () < 0)
         goto bail;
     
@@ -412,14 +396,14 @@
     printf ("LoadFile: %d %d\n", startFrame, stopFrame);
     #endif
     
-    //try {
+    /*try {*/
     
-        // create a new player, and attach to the audio unit
+        /* create a new player, and attach to the audio unit */
         
         thePlayer = new_AudioFilePlayer(ref);
         if (thePlayer == NULL) {
             SDL_SetError ("LoadFile: Could not create player");
-            return -3; //throw (-3);
+            return -3; /*throw (-3);*/
         }
             
         if (!thePlayer->SetDestination(thePlayer, &theUnit))
@@ -431,8 +415,8 @@
         if (stopFrame >= 0 && stopFrame > startFrame)
             thePlayer->SetStopFrame (thePlayer, stopFrame);
         
-        // we set the notifier later
-        //thePlayer->SetNotifier(thePlayer, FilePlayNotificationHandler, NULL);
+        /* we set the notifier later */
+        /*thePlayer->SetNotifier(thePlayer, FilePlayNotificationHandler, NULL);*/
             
         if (!thePlayer->Connect(thePlayer))
             goto bail;
@@ -441,11 +425,11 @@
         thePlayer->Print(thePlayer);
         fflush (stdout);
         #endif
-    //}
-    //catch (...)
-    //{
-    //    goto bail;
-    //}
+    /*}
+      catch (...)
+      {
+          goto bail;
+      }*/
         
     error = 0;
 
@@ -453,16 +437,12 @@
     return error;
 }
 
-//ΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡ
-//  ReleaseFile
-//ΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡ
-
 int ReleaseFile ()
 {
     int error = -1;
         
-    // (Don't see any way that the original C++ code could throw here.) --ryan.
-    //try {
+    /* (Don't see any way that the original C++ code could throw here.) --ryan. */
+    /*try {*/
         if (thePlayer != NULL) {
             
             thePlayer->Disconnect(thePlayer);
@@ -471,22 +451,18 @@
             
             thePlayer = NULL;
         }
-    //}
-    //catch (...)
-    //{
-    //    goto bail;
-    //}
+    /*}
+      catch (...)
+      {
+          goto bail;
+      }*/
     
     error = 0;
     
-//  bail:
+/*  bail: */
     return error;
 }
 
-//ΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡ
-//  PlayFile
-//ΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡ
-
 int PlayFile ()
 {
     OSStatus result = -1;
@@ -494,17 +470,17 @@
     if (CheckInit () < 0)
         goto bail;
         
-//    try {
+    /*try {*/
     
         // start processing of the audio unit
         result = AudioOutputUnitStart (theUnit);
             if (result) goto bail; //THROW_RESULT("PlayFile: AudioOutputUnitStart")
         
-//    }
-//    catch (...)
-//    {
-//        goto bail;
-//    }
+    /*}
+    catch (...)
+    {
+        goto bail;
+    }*/
     
     result = 0;
     
@@ -512,10 +488,6 @@
     return result;
 }
 
-//ΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡ
-//  PauseFile
-//ΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡ
-
 int PauseFile ()
 {
     OSStatus result = -1;
@@ -523,26 +495,22 @@
     if (CheckInit () < 0)
         goto bail;
             
-    //try {
+    /*try {*/
     
-        // stop processing the audio unit
+        /* stop processing the audio unit */
         result = AudioOutputUnitStop (theUnit);
-            if (result) goto bail;  //THROW_RESULT("PauseFile: AudioOutputUnitStop")
-    //}
-    //catch (...)
-    //{
-    //    goto bail;
-    //}
+            if (result) goto bail;  /*THROW_RESULT("PauseFile: AudioOutputUnitStop")*/
+    /*}
+      catch (...)
+      {
+          goto bail;
+      }*/
     
     result = 0;
 bail:
     return result;
 }
 
-//ΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡ
-//  SetCompletionProc
-//ΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡ
-
 void SetCompletionProc (CDPlayerCompletionProc proc, SDL_CD *cdrom)
 {
     assert(thePlayer != NULL);
@@ -552,10 +520,6 @@
     thePlayer->SetNotifier (thePlayer, FilePlayNotificationHandler, cdrom);
 }
 
-//ΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡ
-//  GetCurrentFrame
-//ΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡΡ
-
 int GetCurrentFrame ()
 {    
     int frame;
@@ -578,13 +542,13 @@
     
     OSStatus result = noErr;
     
-    // Create the callback semaphore
+    /* Create the callback semaphore */
     callbackSem = SDL_CreateSemaphore(0);
 
-    // Start callback thread
+    /* Start callback thread */
     SDL_CreateThread(RunCallBackThread, NULL);
 
-    { //try {
+    { /*try {*/
         ComponentDescription desc;
     
         desc.componentType = kAudioUnitComponentType;
@@ -609,10 +573,10 @@
                     
         playBackWasInit = true;
     }
-    //catch (...)
-    //{
-    //    return -1;
-    //}
+    /*catch (...)
+      {
+          return -1;
+      }*/
     
     return 0;
 }
@@ -621,7 +585,7 @@
 {
     if (inStatus == kAudioFilePlay_FileIsFinished) {
     
-        // notify non-CA thread to perform the callback
+        /* notify non-CA thread to perform the callback */
         SDL_SemPost(callbackSem);
         
     } else if (inStatus == kAudioFilePlayErr_FilePlayUnderrun) {
@@ -661,4 +625,4 @@
     return 0;
 }
 
-//}; // extern "C"
+/*}; // extern "C" */