[PATCH v2] null-sink: correct update of timestamp when state changes to RUNNING or IDLE

[tanuk@xxxxxx (Tanu Kaskinen)]

On Sat, 2015-10-24 at 13:00 +0200, Georg Chini wrote:
> u-> timestamp should be reset to "now" when the sink state changes from
> INIT or SUSPENDED to IDLE or RUNNING.
> Thanks to Olivier Delbeke for reporting the bug and testing the patch.
> Bug report is here:
> http://lists.freedesktop.org/archives/pulseaudio-discuss/2015-October/024553.html
> 
> ---
> Â src/modules/module-null-sink.c | 6 ++++--
> Â 1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/src/modules/module-null-sink.c b/src/modules/module-null-sink.c
> index 0270ba2..45532cc 100644
> --- a/src/modules/module-null-sink.c
> +++ b/src/modules/module-null-sink.c
> @@ -93,8 +93,10 @@ static int sink_process_msg(
> Â Â Â Â Â switch (code) {
> Â Â Â Â Â Â Â Â Â case PA_SINK_MESSAGE_SET_STATE:
> Â 
> -Â Â Â Â Â Â Â Â Â Â Â Â if (PA_PTR_TO_UINT(data) == PA_SINK_RUNNING)
> -Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â u->timestamp = pa_rtclock_now();
> +Â Â Â Â Â Â Â Â Â Â Â Â if (pa_sink_get_state(u->sink) == PA_SINK_SUSPENDED || pa_sink_get_state(u->sink) == PA_SINK_INIT) {
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â if (PA_PTR_TO_UINT(data) == PA_SINK_RUNNING || PA_PTR_TO_UINT(data) == PA_SINK_IDLE)
> +Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â u->timestamp = pa_rtclock_now();
> +Â Â Â Â Â Â Â Â Â Â Â Â }
> Â 
> Â Â Â Â Â Â Â Â Â Â Â Â Â break;

Thanks a lot! Pushed to master.

--Â 
Tanu