Re: [fluid-dev] [PATCH] Same expression on both sides of '||'.

[David Henningsson]

On 2015-01-21 19:00, Carlos Olmedo Escobar wrote:
> wakeup_threads_m replaced with thread_ready_m. Spotted thanks to cppcheck.

Applied now. Thanks!

> Signed-off-by: Carlos Olmedo Escobar <address@hidden>
> ---
>   fluidsynth/src/rvoice/fluid_rvoice_mixer.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/fluidsynth/src/rvoice/fluid_rvoice_mixer.c 
> b/fluidsynth/src/rvoice/fluid_rvoice_mixer.c
> index 03a75c7..4672cb8 100644
> --- a/fluidsynth/src/rvoice/fluid_rvoice_mixer.c
> +++ b/fluidsynth/src/rvoice/fluid_rvoice_mixer.c
> @@ -563,7 +563,7 @@ new_fluid_rvoice_mixer(int buf_count, int fx_buf_count, 
> fluid_real_t sample_rate
>     mixer->thread_ready_m = new_fluid_cond_mutex();
>     mixer->wakeup_threads_m = new_fluid_cond_mutex();
>     if (!mixer->thread_ready || !mixer->wakeup_threads ||
> -      !mixer->wakeup_threads_m || !mixer->wakeup_threads_m) {
> +      !mixer->thread_ready_m || !mixer->wakeup_threads_m) {
>       delete_fluid_rvoice_mixer(mixer);
>       return NULL;
>     }