ToneProducer Parameters Change with Delay

[bonefish]

hrev34615, gcc 4

The ToneProducer app from the ​BeOS sample code has controls for three parameters (frequency, amplitude, wave form). Adjusting those controls takes effect with a significant delay (several seconds) on my machine. Since the application uses a parameter web and the parameter changes go through BControllable:{Get,Set}ParameterValue(), I would guess there's some incorrect handling of system vs. performance time. Assuming that the sample code worked correctly under BeOS, that would suggest a bug in Haiku.

For building with gcc 4 the inclusion of <stdlib.h> and <string.h> have to be added to NodeHarnessWin.cpp and stdc++ to the LIBS variable in the makefile.

[aldeck]

I confirm i noticed this problem with the flanger media add-on (included in the image).

[3dEyes]

I think the problem is related to the processing of messages in the queue of the BMediaEventLooper class. When called SetParameterValue(), it creates a message (media_timed_event) with parameters, which is placed in a queue (EventQueue()->AddEvent(ev)). Theoretically, this message should be received in HandleEvent(), but it does not.

[Barrett]

I'm not convinced by the SetValue at line 653, is it correct to pass system_time() here? Might there be need for a little more complex calculus based on current performance time.

[Barrett]

Fixed in hrev50191.

NOTE: In the end I was right, don't use system_time in place of performance time.