Mmmm, sure? There are 3 methods interested by this purpose:

- void SetSoundName(const char* name)
- iSndSysSource* GetSoundSource()
- bool GetSource()

The first and the second are public but the last is private!

If I want the sound source i must call "GetSoundSource()", but the 
current body of this method is (row 143 in "soundfact.h"):

virtual iSndSysSource* GetSoundSource () {
    return source;
  }

No call to "GetSource()" is present in this implementation and the user 
can't call it because it's a private method of "celPcSoundSource"!

So if you need to setup a sound you must add, in my opinion, a call or 
in "SetSoundName(const char* name)" or in "GetSoundSource()" ;)

Greetings, Emanuele

Jorrit Tyberghein ha scritto:
> As I already reported on the bug ticket this is not a bug.
> SetSoundName()'s purpose
> is *only* to set the name of the sound source. Nothing more. The application
> has to call GetSource() to actually setup the sound. So the code is
> correct as it is
> in svn.
>
> Greetings

-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php
_______________________________________________
Cel-main mailing list
Cel-main@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/cel-main

Reply via email to