Thread: Something wrong with my function!?

I have created a function that is supposed to load a *.wav file usig DirectMusic. But the function doesnt pass WCHAR* szWAV corectly. szWAV is suposed to be the filename of my *.wav file...
The program runs without errors but the sound doesn't work.

Why doesn't the soundfile load? When I am not using the function it works fine...?!?!

The code:

void LoadSoundFile(IDirectMusicPerformance8 * PERFORMANCE, IDirectMusicSegment8 * SEGMENT, WCHAR* szWAV)
{
Loader->LoadObjectFromFile(CLSID_DirectMusicSegment,
IID_IDirectMusicSegment8, szWAV, (void **) &SEGMENT);
SEGMENT->SetParam(GUID_DirectMusicAllTypes, 0xFFFFFFFF,
DMUS_SEG_ALLTRACKS, 0, NULL);
SEGMENT->Download(PERFORMANCE);
}

How I'm using the function:

//DirectSound stuff needed...
IDirectMusicPerformance8 * Sound = NULL;
IDirectMusicSegment8 * Gun = NULL;

//Calling the funtion...
LoadSoundFile(Gun, GunSound, L"Gun.wav");

I didnt know where to post this question but I hope someone could help me out.

Thanks for your help //Aram

The program runs and everything its just that the file dosent load so that the buffer is still NULL. I can't figure it out why? I can even misspell the filename and the thing still runs without errors.

By the way it shold be DirectMucis that I am using and not DirectSound...

There's a few things in your post that need to be cleared up.

//Calling the funtion...
LoadSoundFile(Gun, GunSound, L"Gun.wav");

What's the L in there for, should be a typo in your post?



IDirectMusicPerformance8 * Sound = NULL;

Are you initializing the Sound/PERFORMANCE Var with a value other than NULL anywhere?


According to the code you've posted it looks to me that you don't set the PERFORMANCE value to anything other than NULL. So when you code this...

SEGMENT->Download(PERFORMANCE);

...it's pointing to NULL.

OOOOOOOOOORRRRRRRRRRRRRRRRRRRRRRR

I just saw this, why are you calling your Vars different names when calling the function...

LoadSoundFile(Gun, GunSound, L"Gun.wav");

you declared your vars to be "Gun" and "Sound", there's no GunSound.

Plus, they are being called in a different order - shouldn't it be...

LoadSoundFile(Sound, Gun,"Gun.wav");

???????????????????????

Uh oh!!!

Damn I was really tired when I wrote the post but I'll try again.

This is how the whole thing looks like it should be:

//This is a funtion for initializing DirectMusicPerformance.
//it is is the overall manager of music playback

void InitDirectMusic(IDirectMusicPerformance8 * PERFORMANCE)
{
::CoInitialize(NULL);

::CoCreateInstance(CLSID_DirectMusicPerformance, NULL,
CLSCTX_INPROC, IID_IDirectMusicPerformance8,
(void **)&PERFORMANCE);

PERFORMANCE->InitAudio(NULL, NULL, NULL,
DMUS_APATH_DYNAMIC_MONO,
64, DMUS_AUDIOF_ALL, NULL);

}

//used to manage the enumeration and loading objects, it also cache them so that they are not loaded more then once
IDirectMusicLoader8 * Loader;

//Initiating the Loader
void InitLoader()
{
::CoInitialize(NULL);

::CoCreateInstance(CLSID_DirectMusicLoader, NULL,
CLSCTX_INPROC, IID_IDirectMusicLoader8,
(void **)&Loader);
}

//This is the part thats not working corectly.

void LoadSoundFile(IDirectMusicPerformance8 * PERFORMANCE,
IDirectMusicSegment8 * SEGMENT, WCHAR* szWAV)
{
Loader->LoadObjectFromFile(CLSID_DirectMusicSegment,
IID_IDirectMusicSegment8, szWAV, (void **) &SEGMENT);
SEGMENT->SetParam(GUID_DirectMusicAllTypes, 0xFFFFFFFF,
DMUS_SEG_ALLTRACKS, 0, NULL);
SEGMENT->Download(PERFORMANCE);
}

Now when I'm using all of this:

IDirectMusicPerformance8 * Sound = NULL;
IDirectMusicSegment8 * Gun = NULL;

int WINAPI WinMain(HINSTANCE hInstance, HINSTANCE, LPSTR, int)
{
//bla bla for creating the window and so on

InitDirectMusic(Sound);
InitLoader();

//Now loading a soundfile. The L infront is needed or else I would call the ASCII string and not a UNICODE string.

LoadSoundFile(Sound, Gun, L"Gun.wav");

//here is the infinite loop...
}

I could load the file using the regular way like this:

Loader->LoadObjectFromFile(CLSID_DirectMusicSegment,
IID_IDirectMusicSegment8, L"Gun.wav", (void **) &Gun);
Gun->SetParam(GUID_DirectMusicAllTypes, 0xFFFFFFFF,
DMUS_SEG_ALLTRACKS, 0, NULL);
Gun->Download(Sound);

This is how LoadObjectFromFile is declared in the DirectMusic header if its to any help. I havent programed much so I'm not shure if understood it right.

STDMETHOD(LoadObjectFromFile) (THIS_ REFGUID rguidClassID,
REFIID iidInterfaceID,
WCHAR *pwzFilePath,
void ** ppObject) PURE;



I hope I got it right this time...