Starling Forum

1. 

   JaimeDominguez
   

   Expert Bird
   Joined: Dec '11
   Posts: 454

   

   JaimeDominguez

   Expert Bird
   Joined: Dec '11
   Posts: 454

   Since it will bring some extra performance I would like to enable it.

   But first I need to know why when I compile CE with inline functions enabled, the game (in runtime) complains here (CitrusSoundInstance):

   protected static var _maxChannels:uint = SoundChannelUtil.maxAvailableChannels();

   If I bypass it by setting it to =256

   It will complain here:

   protected static var _nonPermanent:Vector.<CitrusSoundInstance> = new Vector.<CitrusSoundInstance>();

   Anyone? Thanks!

   EDIT: Actually I just found out, there was a bug:

   public static function get maxChannels():uint { return _maxChannels; };
    
   		{
   			if(maxChannels < 32)
   				trace("[CitrusSoundInstance] maximum number of concurrent SoundChannels for this instance of CitrusEngine :", maxChannels);
   		}

   Can you spot it?

   Posted 2 years ago #
2. 

   gsynuh
   

   Citrus Jedi
   Joined: Nov '12
   Posts: 2,687

   

   gsynuh

   Citrus Jedi
   Joined: Nov '12
   Posts: 2,687

   I can't see it :/

   after initializing _maxChannels, this is just to trace a warning if we don't have the full 32 channels available at the start of the app. I shouldn't've used this form of static initializer then - didn't test inline with it - I guess SoundManager could have a look at CitrusSoundInstance.maxChannels and trace that warning itself.

   Posted 2 years ago #
3. 

   JaimeDominguez
   

   Expert Bird
   Joined: Dec '11
   Posts: 454

   

   JaimeDominguez

   Expert Bird
   Joined: Dec '11
   Posts: 454

   Nop!
   After the get funcion which finishes in the same line. there is some unlinked code in brackets that doesn't do anything ... Does it?

   Posted 2 years ago #
4. 

   gsynuh
   

   Citrus Jedi
   Joined: Nov '12
   Posts: 2,687

   

   gsynuh

   Citrus Jedi
   Joined: Nov '12
   Posts: 2,687

   Yeah it's just a warning but it does work. I could put it in a more conventional place and it wouldn't make much of a difference I guess.

   edit : http://kaioa.com/node/100

   edit 2 : just wanted this to happen as soon as maxChannels was declared, and I didn't think it was an important piece of code enough to have to include code someplace else - but I guess I should because it might now be considered bad as3 practice - specially if this breaks inline obviously

   Posted 2 years ago #
5. 

   gsynuh
   

   Citrus Jedi
   Joined: Nov '12
   Posts: 2,687

   

   gsynuh

   Citrus Jedi
   Joined: Nov '12
   Posts: 2,687

   fixed (about time right?)

   Posted 8 months ago #

RSS feed for this topic

Topic Closed

This topic has been closed to new replies.