[MPlayer-cvslog] r29979 - trunk/mplayer.c

[Diego Biurrun]

On Tue, Dec 08, 2009 at 12:11:40AM +0100, reimar wrote:
> 
> Log:
> Add braces to avoid ambiguous else.
> 
> --- trunk/mplayer.c	Mon Dec  7 04:52:55 2009	(r29978)
> +++ trunk/mplayer.c	Tue Dec  8 00:11:40 2009	(r29979)
> @@ -2445,11 +2445,12 @@ static void pause_loop(void)
>      mpctx->osd_function=OSD_PLAY;
> -    if (mpctx->audio_out && mpctx->sh_audio)
> +    if (mpctx->audio_out && mpctx->sh_audio) {
>          if (mpctx->eof) // do not play remaining audio if we e.g.  switch to the next file
>            mpctx->audio_out->reset();
>          else
>            mpctx->audio_out->resume(); // resume audio
> +    }
>      if (mpctx->video_out && mpctx->sh_video && vo_config_count)
>          mpctx->video_out->control(VOCTRL_RESUME, NULL); // resume video

I thought the convention was that an else always corresponds to the last if?

Diego