AL_INVALID_VALUE error when loading the first sound of the game about solarus HOT 4 CLOSED

Actually it's the first call to alGenBuffers() on the OSX implementation which fail. (since OpenAl 1.1 according to my following research)

A simple call with NULL buffer at the begining of Sound::load_all() makes the error disappear.

I don't know why cleaning OpenAl error buffer before Sound::load() (and then just after construct it) make disappear the error too, and I also don't know how to solve it properly ;)

from solarus.

I can accept this kind of workaround. I implemented it in commit 35120c3 (branch master). Does it work?
Feel free to make a pull request to fix it if it's not okay.

I don't want to make the call with the NULL buffer from Sound::load_all() because Sound::load_all() is not necessarily called.

from solarus.

Works fine for me :)

Because of the high number of OSX's (or not) workaround, what do you think about manage them by adding some SOLARUS_WORKAROUND_XXX definition in the CMakeList file ?
Not really useful (unless there is an huge commit on all used lib which fix every bug) but it can provide better vision of the OS's dependent code when all "strange issue" will be solved.

EDIT : The patch I just submitted propose an implementation. Accept it if this solution suits you ;)

from solarus.

Hi,
I applied a part of your changes, and added new compilation flags to remove some #ifdef APPLE_, #ifdef __PANDORA, etc.
I removed both workarounds because it's not a problem to apply them in all cases. The less OS-specific code, the better.

Thanks for your patch!

from solarus.