cocos2d for iPhone

A fast, easy to use, free, and community supported 2D game engine

1. 

   adik
   Member

   Hello everybody,

   I am having problems with unloading my sound effects. My game consists of several levels. In each level I do all my sound effect preloading on its init method using SimpleAudioEngine's preloadEffect and unload level specific sound effects on its dealloc method using unloadEffect.
   I get this error on my console output, (with NSZombieEnabled=YES):
   *** -[CFNumber intValue]: message sent to deallocated instance 0xe619580

   my call stack looks like this:
   

   instruments:
   

   full console output

   This usually occurs when I play 4-5 levels in sequence. Then at the end of 5th level it crashes. If I start playing from that 5th level, the crash never occurs. Would appreciate your help. Thank you.

   A.

   Posted 2 years ago #
2. 

   Steve Oldmeadow
   Administrator

   It looks like a bug. Can you step through unloadEffect in the debugger and work out what the problem is?

   Posted 2 years ago #
3. 

   adik
   Member

   I have identified the problem.
   
//SimpleAudioEngine.m

   -(void) unloadEffect:(NSString*) filePath
   {
   NSNumber* soundId = [loadedEffects objectForKey:filePath];
   if(soundId == nil)
   {
   #ifdef ASSERT_DEBUG
   @throw [[[NSException alloc] initWithName:@"SimpleAudioEngine::unloadEffect" reason:filePath userInfo:nil] autorelease];
   #else
   return;
   #endif
   }
   DLog("UNLOADING EFFECT => SoundId: %d, Name: %@", [soundId intValue], filePath);
   [self freeBuffer:soundId];

   /* original
   [loadedEffects removeObjectForKey:filePath]; // <- here soundId gets autoreleased
   [soundEngine unloadBuffer:[soundId intValue]]; // <- here we are referencing already released object
   */

   // fix
   [soundEngine unloadBuffer:[soundId intValue]];
   [loadedEffects removeObjectForKey:filePath];
   }

   My quick solution was simply to swap those two lines of code, so that sound buffer gets unloaded first and then it gets removed from loadedEffects NSMutableDictionary.
   But what still remains a mystery to me is why it didn't throw an exception on any other levels when I do not play in sequence. Maybe I will look at it when I have more time.

   Posted 2 years ago #
4. 

   shadaz
   Member

   Hello,

   In testing my app with iphone 4, I ran across this exact same problem. So I searched the forums to see if anyone else had this issue before. I implemented adik's fix and it seemed to have solved it.

   has a bug been filed for this issue?

   thank you adik!

   Posted 1 year ago #
5. 

   Steve Oldmeadow
   Administrator

   No bug has been filed because that code doesn't exist any more. You must be using an old version of CocosDenshion?

   The version of CocosDenshion in 0.99.4 RC 3 is the one you should be using.

   Posted 1 year ago #
6. 

   shadaz
   Member

   I tried to update to .99.4 rc3 today to fix other iphone 4 issues, but all my particle routines blew up and the ones I got working either look totally different or ran at like 1 fps. So I had to revert back to the code I originally built the project with (.99.2), it is then during testing that I saw this problem. Im glad it is fixed in latest release, as any new projects I make will be on the new code, its just hard for me to get my old project working with the new codebase. thanks steve.

   Posted 1 year ago #

RSS feed for this topic

Reply

You must log in to post.