[vlc-devel] [vlc-commits] aout: give volume/mute change a chance after DecPlay (refs #8240)

[Rafaël Carré]

Hello,

Le 27/02/2013 21:18, Rémi Denis-Courmont a écrit :
> vlc/vlc-2.0 | branch: master | Rémi Denis-Courmont <remi at remlab.net> | Wed Feb 27 22:16:56 2013 +0200| [c14c1b137c0b9d191993ecbc46d307022399ad78] | committer: Rémi Denis-Courmont
> 
> aout: give volume/mute change a chance after DecPlay (refs #8240)
> 
> This is a quick uninvasive work-around for #8240.
> A more involved fix is in f788153763bf56fda744df5b9abaa893c402378e.
> 
>> http://git.videolan.org/gitweb.cgi/vlc/vlc-2.0.git/?a=commit;h=c14c1b137c0b9d191993ecbc46d307022399ad78
> ---
> 
>  src/audio_output/dec.c |    6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/src/audio_output/dec.c b/src/audio_output/dec.c
> index 3c77723..5b720f3 100644
> --- a/src/audio_output/dec.c
> +++ b/src/audio_output/dec.c
> @@ -29,6 +29,11 @@
>  #endif
>  
>  #include <assert.h>
> +#ifdef __unix__

Is that meant to include OSX?

It doesn't and the macro clashes with the definition (sched.h is brought
up by pthread.h)

Perhaps POSIX define would be more appropriate?

> +# include <sched.h>
> +#else
> +# define sched_yield() (void)0
> +#endif
>  
>  #include <vlc_common.h>
>  #include <vlc_aout.h>
> @@ -333,6 +338,7 @@ int aout_DecPlay (audio_output_t *p_aout, block_t *p_buffer, int i_input_rate)
>      }
>  
>      aout_unlock( p_aout );
> +    sched_yield();
>      return 0;
>  }