Re: [Super-tux-devel] Sound and sound again :-) -> Bill

[Duong-Khang N. <neo...@us...>]

>
> On Thu, Dec 25, 2003 at 09:03:18PM +0100, Duong-Khang NGUYEN wrote:
> > Changes,fixes, adds:
> >
> > 1. Changes: setup.c now open audio device even if --disable-sound
> > command-line option was given. This is because the user can now decide to
> > activate it later via menu.
>
> I think maybe if they use "--disable-sound", it shouldn't open the audio
> device, and either it should be opened the first time they enable sound
> from the menu, OR the menu option would just not be there.
>
> Then perhaps have a "--no-sound" or "--sound=off" option which changes
> the OPTION (rather than affect the core initialization)
>
> Make sense?
>

Currently, the audio device is initialized only once at the startup by 
"setup.c" and this regarless the command-line options "--disable-sound" or 
"--disable-music". If both of them are specified, "setup.c" still tries to 
initialized the audio device, but DOES NOT print any error message like:

	"Warning: I could not set up audio for 44100 Hz 16-bit stereo.
	The Simple DirectMedia error that occured was:
	No available audio device"

In fact, the command-line options "--disable-sound" and "--disable-music" only 
affect the menu options. And at a higher level, they're completly IGNORED and 
sound/music menu options are DISABLED if the "setup.c" can not initialize the 
audio device at the beginning. IOW, "--disable-sound" and "--disable-music" 
don't affect the core initialization (core initialization = open and init the 
audio device via SDL_Init and Mix_OpenAudio)

Let me know if you have any other ideas.

Duong-Khang NGUYEN


-- 
Never say if I could turn back the time,
life is going on !