I call it memory leak, you can call it whatever you want

I'm making my first game for a school project but I have an Issue managing the Audio side.

Everytime I call Shoot, it starts the 1 second clip but it stays in the memory. The only way I can avoid this so far is to do the Clip.Close() but thats going to stop the audio right after starting it so it doesnt help.



SoundManager class

1. use a line listener
2. if #1 doesn't work, the more general answer is to use a patterns to treat asynchronous methods as synchronous which solves this problem. alternatively, you can use a queue. create a wrapper for the clip so you can store when it started (or does it have a state that tells you when it's done?). when you start the clip, add to the queue. have a separate thread look at the queue and close/dequeue when enough time passes
3. if you "shoot" faster than the duration of the clip, what should happen? if you only want one sound file to run at any given time, i recommend keeping an object-level variable instead of recreating it every time you shoot.
4. you should have a single static logger at the class level. dont keep redeclaring it every time you use it.

This post was edited by carteblanche on Jun 3 2012 07:19pm

thanks you for the reply. I was going to try line #1 but im going to get OpenAL instead More tutorials and some cool features.

Maybe you can give more memory to the virtual machine so you can avoid that leak of it.

???